Fix some comment and code typos

I release these contributions under the terms of the Imazen Contributor
Agreement V1.0 (http://imageresizing.net/licenses/contribute), the terms
of which I accept and agree.
- @stakx, 21 Jan 2018

Author: stakx

Diff for: CONTRIBUTING.md

@@ -13,6 +13,6 @@ Small commits can simply include this statement:
         (http://imageresizing.net/licenses/contribute), the terms of which I accept and agree.
         - [Name] [Date]
 
-Frequent or heavy commiters should [sign and date the actual agreement](http://imageresizing.net/licenses/contribute), which only needs to be done once. Digital signatures or scanned copies are accepted; send them to [email protected]. 
+Frequent or heavy committers should [sign and date the actual agreement](http://imageresizing.net/licenses/contribute), which only needs to be done once. Digital signatures or scanned copies are accepted; send them to [email protected]. 
 
 If you are introducing any new dependencies, those licenses may obligate you to create a notice.txt or license.txt file.

Diff for: README.md

@@ -29,7 +29,7 @@ You can use command strings that are compatible with [ImageResizer 4 querystring
 
 Or submit a JSON job file. JSON jobs can have multiple inputs and outputs, and can represent any kind of operation graph.
 
-The following generates mutiple sizes of an image from an example job file:
+The following generates multiple sizes of an image from an example job file:
 
 ```
 imageflow_tool v0.1/build --json examples/export_4_sizes/export_4_sizes.json
@@ -43,7 +43,7 @@ imageflow_tool v0.1/build --json examples/export_4_sizes/export_4_sizes.json
 
 By default, imageflow_tool prints a JSON response to stdout. You write this to disk with `--response`.
 
-`--debug-package` will create a .zip file to reproduce problematic behavior with both `v0.1/build` and `v0.1/ir4`. Please sumbit bug reports; we try to make it easy.
+`--debug-package` will create a .zip file to reproduce problematic behavior with both `v0.1/build` and `v0.1/ir4`. Please submit bug reports; we try to make it easy.
 
 ## Using imageflow_server for dynamic imaging
 
@@ -64,7 +64,7 @@ Now you can edit images from HTML... and use srcset without headache.
 
 ### Beyond the demo
 
-You'll want to mount varous image source locations to prefixes. The `--mount` command parses a colon (`:`) delimited list of arguments. The first is the prefix you'll use in the URL (like `http://localhost:39876/prefix/`. The second is the engine name. Remaining arguments are sent to the engine.
+You'll want to mount various image source locations to prefixes. The `--mount` command parses a colon (`:`) delimited list of arguments. The first is the prefix you'll use in the URL (like `http://localhost:39876/prefix/`. The second is the engine name. Remaining arguments are sent to the engine.
 
 #### Examples
 

Diff for: bindings/ruby/lib/imageflow/vips/benchmark.rb

@@ -240,7 +240,7 @@ def benchmark!(log:)
             log << "\nComparing against #{reference}\n"
             png_images.each do |output|
               log << `dssim #{reference} #{output}`
-              log << "Viusalize with > compare #{reference} #{output} -fuzz 1%  x:\n"
+              log << "Visualize with > compare #{reference} #{output} -fuzz 1%  x:\n"
             end
           end
         end

Diff for: c_components/imageflow.h

@@ -59,7 +59,7 @@ typedef enum flow_codec_type {
 
 typedef enum flow_scanlines_filter_type {
     flow_scanlines_filter_Sharpen, // 3x3, percentage-based
-    flow_scanlines_filter_Blur, // 3x box blur to simulate guassian
+    flow_scanlines_filter_Blur, // 3x box blur to simulate Gaussian
     flow_scanlines_filter_Convolve, // Apply convolution kernel
     flow_scanlines_filter_ColorMatrix, // Apply color matrix
     flow_scanlines_filter_ToLinear,

Diff for: c_components/imageflow_advanced.h

@@ -272,7 +272,7 @@ typedef double (*flow_detailed_interpolation_method)(const struct flow_interpola
 struct flow_interpolation_details {
     // 1 is the default; near-zero overlapping between windows. 2 overlaps 50% on each side.
     double window;
-    // Coefficients for bucubic weighting
+    // Coefficients for bicubic weighting
     double p1, p2, p3, q1, q2, q3, q4;
     // Blurring factor when > 1, sharpening factor when < 1. Applied to weights.
     double blur;
@@ -322,7 +322,7 @@ struct flow_interpolation_pixel_contributions {
     float * Weights; /* Normalized weights of neighboring pixels */
     int Left; /* Bounds of source pixels window */
     int Right;
-}; /* Contirbution information for a single pixel */
+}; /* Contribution information for a single pixel */
 
 struct flow_interpolation_line_contributions {
     struct flow_interpolation_pixel_contributions * ContribRow; /* Row (or column) of contribution weights */

Diff for: docs/boxstarter

@@ -1,4 +1,4 @@
-#boxstarter really didn't work for me. I'd suggest using chocolatey directly and changing ExlorerOptions yourself.
+#boxstarter really didn't work for me. I'd suggest using chocolatey directly and changing ExplorerOptions yourself.
 
 
 # Configure Windows

Diff for: docs/redis.md

@@ -4,9 +4,9 @@
 
 * volatile-ttl allows us to confidently store metadata for remote objects, but also utilize leftover space for byte or bitmap data caching
 * Persistence required, but interval is less crucial. 
-* maxmemory-policy to noeviction will block writes instead of exhasuting resources.
+* maxmemory-policy to noeviction will block writes instead of exhausting resources.
 
-Partitioning across Redis servers needs to happen by an inexpensively-computed key. For RIAPI requests this could be the 'primay image' base URL. For JSON requests this could be the first listed resource. Duplicates would happen for multi-image sources.
+Partitioning across Redis servers needs to happen by an inexpensively-computed key. For RIAPI requests this could be the 'primary image' base URL. For JSON requests this could be the first listed resource. Duplicates would happen for multi-image sources.
 
 ## Schema:
 
@@ -107,7 +107,7 @@ Metadata always goes in when encountered. Blobs have admission criteria
 
 * Second request?
 * Sliding scale based on recent evictions?
-* Request/compuation cost vs. storage container retreival cost
+* Request/computation cost vs. storage container retrieval cost
 
 
 

Diff for: imageflow_abi/src/lib.rs

@@ -752,7 +752,7 @@ pub extern "C" fn imageflow_context_get_output_buffer_by_id(context: *mut Contex
         let s = c.get_output_buffer_slice(io_id).map_err(|e| e.at(here!()))?;
 
         if s.len().leading_zeros() == 0 {
-            Err(nerror!(ErrorKind::Category(ErrorCategory::InternalError), "Error retriving output buffer; length overflow prevented (most significant bit set)."))
+            Err(nerror!(ErrorKind::Category(ErrorCategory::InternalError), "Error retrieving output buffer; length overflow prevented (most significant bit set)."))
         } else {
             unsafe {
                 (*result_buffer) = s.as_ptr();
@@ -831,7 +831,7 @@ pub fn exercise_json_message() {
         let expected_response = c::JsonResponse::teapot();
         let expected_json_out = ::std::str::from_utf8(
             expected_response.response_json.as_ref()).unwrap();
-        let expected_reponse_status = expected_response.status_code;
+        let expected_response_status = expected_response.status_code;
 
         let response = imageflow_context_send_json(c,
 
@@ -856,7 +856,7 @@ pub fn exercise_json_message() {
             ::std::str::from_utf8(std::slice::from_raw_parts(json_out_ptr, json_out_size)).unwrap();
         assert_eq!(json_out_str, expected_json_out);
 
-        assert_eq!(json_status_code, expected_reponse_status);
+        assert_eq!(json_status_code, expected_response_status);
 
         imageflow_context_destroy(c);
     }

Diff for: imageflow_core/src/clients/stateless.rs

@@ -1,5 +1,5 @@
 //!
-//! This module provides a thin wrapper over job building and image info retreival.
+//! This module provides a thin wrapper over job building and image info retrieval.
 //! It catches panics and reports them as part of a very simple error enum.
 //!
 //! It would be nice for this to go away or be merged with Context
@@ -100,7 +100,7 @@ impl LibClient {
             Ok(Ok(v)) => Ok(v)
         };
 
-        context.destroy()?; // Termination errors trump exectuion errors/panics
+        context.destroy()?; // Termination errors trump execution errors/panics
         result
 
     }

Diff for: imageflow_core/src/codecs/gif/mod.rs

@@ -125,7 +125,7 @@ impl Decoder for GifDecoder {
             // Grab a reference
             let frame = self.next_frame.as_ref().ok_or_else(|| nerror!(ErrorKind::InvalidOperation, "read_frame was called without a frame available"))?;
 
-            //Prepare our resuable buffer
+            //Prepare our reusable buffer
             let buf_size = self.reader.width() as usize * self.reader.height() as usize;
 
             let buf_mut = self.buffer.get_or_insert_with(|| vec![0; buf_size]);

Diff for: imageflow_core/src/errors.rs

@@ -1100,7 +1100,7 @@ impl CErrorProxy{
     ///  `imageflow_context_clear_error`. You'll be ignored, as will future
     ///   `imageflow_add_to_callstack` invocations.
     ///
-    /// * If you provide an error code of zero one will be substited for you.
+    /// * If you provide an error code of zero one will be substituted for you.
     ///
     /// Returns None if the context already has an error
     fn raise_error(&mut self, e: FlowError)

Diff for: imageflow_core/src/ffi.rs

@@ -17,7 +17,7 @@ use internal_prelude::works_everywhere::*;
 
 // These are reused in the external ABI, but only as opaque pointers
 ///
-/// `ImaeflowJsonResponse` contains a buffer and buffer length (in bytes), as well as a status code
+/// `ImageflowJsonResponse` contains a buffer and buffer length (in bytes), as well as a status code
 /// The status code can be used to avoid actual parsing of the response in some cases.
 /// For example, you may not care about parsing an error message if you're hacking around -
 /// Or, you may not care about success details if you were sending a command that doesn't imply
@@ -386,7 +386,7 @@ pub struct BitmapFloat {
     pixels_borrowed: bool,
     /// The number of floats in the buffer
     float_count: uint32_t,
-    /// The number of floats betwen (0,0) and (0,1)
+    /// The number of floats between (0,0) and (0,1)
     float_stride: uint32_t,
 
     /// If true, alpha has been premultiplied

Diff for: imageflow_core/tests/common/mod.rs

@@ -518,7 +518,7 @@ pub fn compare_bitmaps(_c: &ChecksumCtx,  actual: &mut BitmapBgra, expected: &mu
     }
 }
 
-/// Evalutates the given result against known truth, applying the given constraints
+/// Evaluates the given result against known truth, applying the given constraints
 pub fn compare_with<'a, 'b>(c: &ChecksumCtx, expected_checksum: &str, expected_bitmap : &'b mut BitmapBgra, result: ResultKind<'a>, require: Constraints, panic: bool) -> bool{
     if !check_size(&result, require.clone(), panic) {
         return false;
@@ -562,7 +562,7 @@ pub fn check_size<'a>( result: &ResultKind<'a>, require: Constraints, panic: boo
 
 
 
-/// Evalutates the given result against known truth, applying the given constraints
+/// Evaluates the given result against known truth, applying the given constraints
 pub fn evaluate_result<'a>(c: &ChecksumCtx, name: &str, mut result: ResultKind<'a>, require: Constraints, panic: bool) -> bool{
     let (exact, trusted) = result.exact_match_verbose(c, name);
 

Diff for: imageflow_core/tests/skip/graph.rs

@@ -7,7 +7,7 @@
 //// This test case helped expose a flaw in graph creation, where we swapped max_edges and max_nodes and caused memory
 //// overlap
 //// It also showed how that post_optimize_flatten calls which create pre_optimize_flattenable nodes
-//// Can cause execution to fail in fewer than 6 passes. We may want to re-evaluate our graph exeuction approach
+//// Can cause execution to fail in fewer than 6 passes. We may want to re-evaluate our graph execution approach
 //flow_c * c = flow_context_create();
 //struct flow_graph * g = flow_graph_create(c, 10, 10, 200, 2.0);
 //ERR(c);

Diff for: imageflow_core/tests/skip/malloc.rs

@@ -3,7 +3,7 @@
 // Test malloc higher than swap, reduce until success
 // Write zeroes to huge malloc, verify process doesn't crash (hey, overcommit isn't what you think)
 // Free
-// Malloc 1mb chunks until they fail. See if smaller mallocs succeed, and how many of them. Wait a bit, then try an extra 1mb chunck
+// Malloc 1mb chunks until they fail. See if smaller mallocs succeed, and how many of them. Wait a bit, then try an extra 1mb chunk
 // Have time limit
 
 

Diff for: imageflow_helpers/src/debug.rs

@@ -7,7 +7,7 @@ use std::thread;
 
 use std::sync::{Once, ONCE_INIT};
 
-static CONIDITIONAL_SET: Once = ONCE_INIT;
+static CONDITIONAL_SET: Once = ONCE_INIT;
 static SET_HOOK: Once = ONCE_INIT;
 
 
@@ -21,7 +21,7 @@ pub fn backtraces_wanted() -> bool{
     }
 }
 pub fn upgrade_panic_hook_once_if_backtraces_wanted(){
-    CONIDITIONAL_SET.call_once(|| {
+    CONDITIONAL_SET.call_once(|| {
         if backtraces_wanted(){
             set_panic_hook_once();
         }

Diff for: imageflow_helpers/src/pollster.rs

@@ -1,5 +1,5 @@
 // Single thread
-// shared datastructure for adding endpoints
+// shared data structure for adding endpoints
 // Endpoints callback
 use ::preludes::from_std::*;
 use ::std;

Diff for: imageflow_riapi/src/sizing_tests.rs

@@ -629,7 +629,7 @@ fn test_steps() {
     let mut temp = Vec::new();
     let mut source_sizes = Vec::new();
 
-    //Holds resuable maps/vectors for collecting info
+    //Holds reusable maps/vectors for collecting info
     let mut current = invalid_group_data();
 
     let mut failed_kits = Vec::new();

Diff for: imageflow_server/src/config.rs

@@ -27,7 +27,7 @@
 // pub enum BlobSource{
 //    Directory(String),
 //    HttpServer(String),
-//    //TODO: Azure and S3 blob backens
+//    //TODO: Azure and S3 blob backend
 // }
 //
 // pub enum InternalCachingStrategy{

Diff for: imageflow_types/src/lib.rs

@@ -340,7 +340,7 @@ pub enum CommandStringKind{
     ImageResizer4
 }
 
-/// Constraint types. TODO: expand to include nearly everthing RIAPI does.
+/// Constraint types. TODO: expand to include nearly everything RIAPI does.
 #[derive(Serialize, Deserialize, Clone, PartialEq, Debug)]
 pub enum Constraint {
     #[serde(rename="within")]
@@ -671,7 +671,7 @@ pub struct Build001Config {
 }
 
 /// Represents a complete build job, combining IO objects with a framewise operation graph.
-/// TODO: cleaup builder_config.
+/// TODO: cleanup builder_config.
 #[derive(Serialize, Deserialize, Clone, PartialEq, Debug)]
 pub struct Build001 {
     pub builder_config: Option<Build001Config>,