Fix typos in comments (#1243)

Ran a spell checker on our comments. So many fixes.

Signed-off-by: Larry Gritz <[email protected]>

Author: lgritz

ASWF/project_intake.md

@@ -4,15 +4,15 @@ This is a checklist for TSC's to review as part of the intake process. The TSC s
 
 - Existing Project Governance
   - [X] Project License exists ( [LICENSE.md](../LICENSE.md) ) and is OSI-approved
-  - [X] Any third-party components/dependencies included are listed along with thier licenses ( [THIRD_PARTY.md](../THIRD_PARTY.md) )
-  - [X] Governamce defined, outlining community roles and how decsions are made ( [GOVERNANCE.md](../GOVERNANCE.md] )
+  - [X] Any third-party components/dependencies included are listed along with their licenses ( [THIRD_PARTY.md](../THIRD_PARTY.md) )
+  - [X] Governance defined, outlining community roles and how decsions are made ( [GOVERNANCE.md](../GOVERNANCE.md] )
     - Closed by https://github.com/imageworks/OpenShadingLanguage/pull/1137
   - [X] Contribution Policy defined ( [CONTRIBUTING.md](../CONTRIBUTING.md) )
     - Closed by https://github.com/imageworks/OpenShadingLanguage/pull/1137
   - [X] Code of Conduct defined ( existing or pull from [ASWF Sample Project](https://github.com/AcademySoftwareFoundation/aswf-sample-project/blob/master/CODE_OF_CONDUCT.md) )
     - Closed by https://github.com/imageworks/OpenShadingLanguage/pull/1137
   - [ ] Release methodology defined	( [release.md](process/release.md) )
-- New Project Goverance
+- New Project Governance
   - [X] TSC members identified
   - [X] First TSC meeting held
   - [X] TSC meeting cadence set and added to project calendar
@@ -38,7 +38,7 @@ This is a checklist for TSC's to review as part of the intake process. The TSC s
 	- [X] Logo(s)	( https://github.com/AcademySoftwareFoundation/artwork/blob/master/examples/incubating.md#open-shading-language-logos )
 	- [ ] Trademarks/mark ownership rights ( John to follow up with Larry on this )
 - Outreach
-  - [X] New project annoucement done ( https://www.aswf.io/osl-joins-aswf/ )
+  - [X] New project announcement done ( https://www.aswf.io/osl-joins-aswf/ )
   - [X] Project added to ASWF website and ASWF landscape
 - Graduation
   - [ ] CII Badge	achieved

CHANGES.md

@@ -146,7 +146,7 @@ API changes, new options, new ShadingSystem features (for renderer writers):
   system attribute) now defaults to 1, not 0. That should be the best
   performance while still having reasonable JIT times. #1114 (1.11.4)
 * We no longer automatically build the MaterialX shaders (you need to set
-  build option OSL_BUILD_MATRIALX to ON). This will eventually be deprecated
+  build option OSL_BUILD_MATERIALX to ON). This will eventually be deprecated
   entirely. #1136 (1.11.5)
 * Helper varieties of ShadingSystem::Parameter() and ReParameter() that
   handle the common cases of a single float, int, or string. #1195 #1196
@@ -186,7 +186,7 @@ Performance improvements:
 * Range checking of matrix component access is skipped for certain cases
   where it was unnecessary. #1174 (1.11.6)
 * LLVM JIT+optimize time has been sped up around 10% by doing a better job
-  of pruning unused functions and symbols in the generated modul. #1172
+  of pruning unused functions and symbols in the generated module. #1172
   (1.11.6)
 * `ShadingSystem::symbol_address()` no longer checks for the symbol to be
   NULL (though it will assert on a debug build). #1237 (1.11.7.1)
@@ -232,13 +232,13 @@ Bug fixes and other improvements (internals):
 Internals/developer concerns:
 * Switch much of the internals where we do string formatting using printf-like
   conventions to errorf(), etc., naming convention, reserving the non-f
-  names for future use with C++20 std::format/python convenetions. #1076
+  names for future use with C++20 std::format/python conventions. #1076
   (1.11.3)
 * Remove pointless assertions from the TypeSpec class. #1079
 * Assertion overhaul: use OSL_ASSERT/OSL_DASSERT instead of borrowing from
   OIIO's dassert.h. Note that OSL_ASSERT always tests, prints error if the
   test fails, but then only aborts for debug builds (no abort for release
-  builds). In ocntrast, OSL_DASSERT does all for debug builds, and none
+  builds). In contrast, OSL_DASSERT does all for debug builds, and none
   (not even the test) for release builds. #1080 #1087 (1.11.3)
 * Refactor the implementation of the `Dual` template to support more
   efficient code generation when used in vectorized loops (this will be
@@ -282,7 +282,7 @@ Build & test system improvements:
     - The ENABLERTTI build option has been renamed `ENABLE_RTTI`.
     - Config based install and usage.
 * testshade:
-    - Make sure that no unreported errors accumulted in the texture system
+    - Make sure that no unreported errors accumulated in the texture system
       or image cache. #939 (1.11.0)
     - Check that no leftover errors are in the TextureSystem or ImageCache
       at the end of the test (that would indicate that someplace in OSL we
@@ -330,7 +330,7 @@ Build & test system improvements:
   #956 (1.10.0)
 * When OptiX mode is enabled (USE_OPTIX=1), the testsuite will some tests in
   an additional time forcing OptiX mode: any test containing a file named
-  `OPTIX` in its test directory; or all tests if enviroment variable
+  `OPTIX` in its test directory; or all tests if environment variable
   `TESTSUITE_OPTIX=1` (except tests that contain a file named `NOOPTIX` in
   their directory. #1004 (1.10.0)
 * Allow OSL to build with `USE_LLVM_BITCODE` enabled on Windows. #998 (1.10.0)
@@ -351,7 +351,7 @@ Build & test system improvements:
 * oslconfig.h is now generated by a cmake configure_file step, from
   oslconfig.h.in. #1141 (1.11.5)
 * Use ASWF docker images to speed up many of the CI matrix entries, as well
-  as to excactly test several VFX Platform specs. #1169 (1.11.6)
+  as to exactly test several VFX Platform specs. #1169 (1.11.6)
 * Fix typo that botched the version in the .pc pkgconfig file. #1168 (1.11.6)
 * CI test against gcc9 and gcc10. #1192 (1.11.6)
 * Propagate RTTI compile options to exported cmake config. #1194 (1.11.6)
@@ -621,7 +621,7 @@ Shaders:
     * mandelbrot.osl: computes Mandelbrot and Julia images. #827 (1.10.0)
 * MaterialX support:
     * Improved support for MaterialX 1.36: add sin, cos, tan, atan2, ceil,
-      sqrt, exp, determinent, ln, transpose, sign, rotate, transforms,
+      sqrt, exp, determinant, ln, transpose, sign, rotate, transforms,
       rgb/hsv convert, extract, separate, tiledimage. Rename exponent ->
       power, pack -> combine, hsvadjust -> hueshift. Add some color2/4
       mutual conversion operators. Fixes to ramp4, clean up texture mapping
@@ -679,7 +679,7 @@ API changes, new options, new ShadingSystem features (for renderer writers):
 Bug fixes and other improvements (internals):
 * The context's `texture_thread_info` is now properly passed to the
   RenderServices callbacks instead of passing NULL. (1.10.0)
-* Symbols are enbled in the JIT, allowing Intel Vtune profiler to correctly
+* Symbols are enabled in the JIT, allowing Intel Vtune profiler to correctly
   report which JITed OSL code is being executed. #830 (1.10.0)
 * ASTNode and OSLCompilerImpl -- add info() and message() methods to
   complement the existing error and warning. #854 (1.10.0)
@@ -760,7 +760,7 @@ Release 1.9.10 -- 1 Sep 2018 (compared to 1.9.9)
 * Add more functions to color2.h, color4.h, vector2.h, vector4.h: ceil,
   sqrt, exp, log2, log, sign, sin, cos, tan, asin, acos, atan2. #903, #904
 * Improved support for MaterialX 1.36: add sin, cos, tan, atan2, ceil, sqrt,
-  exp, determinent, ln, transpose, sign, rotate, transforms, rgb/hsv convert,
+  exp, determinant, ln, transpose, sign, rotate, transforms, rgb/hsv convert,
   extract, separate, tiledimage. Rename exponent -> power, pack -> combine,
   hsvadjust -> hueshift. Add some color2/4 mutual conversion operators.
   Fixes to ramp4, clean up texture mapping nodes, fixes to triplanarprojection
@@ -845,7 +845,7 @@ Standard library additions/changes:
 * `smoothstep()` has been extended to `T smoothstep(T edge0, T edge1, T x)`
   for T of any the `triple` types (previously, `smoothstep` only came in
   the `float` variety). #765 (1.9.0/1.8.10)
-* `mix()` has been extenended to support
+* `mix()` has been extended to support
       `color closure mix (color closure A, color closure B, color x)`
   #766 (1.9.0/1.8.10)
 * `hashnoise()` is like cellnoise (1D, 2D, 3D, or 4D input, 1D or 4D output
@@ -1042,7 +1042,7 @@ Developer goodies:
   This lets you use the `Dual` templates for automatic differentiation of
   of ordinary 1D functions (e.g., `Dual<float,1>`) or 3D volumetric
   computations (e.g., `Dual<float,3>`). Most of OSL internals use automatic
-  differentiation on 2 dimenstions (`Dual<float,2>` a.k.a. `Dual2<float>`),
+  differentiation on 2 dimensions (`Dual<float,2>` a.k.a. `Dual2<float>`),
   but this change makes `dual.h` a lot more useful outside of OSL.
   #803 (1.9.1)
 
@@ -1182,7 +1182,7 @@ Language features:
   nested `{ {...}, {...} }`.  #640 (1.8.0)
 * oslc (and liboslcomp) now supports UTF-8 filenames, including on
   Windows. #643 (1.8.0)
-* osl now accepts hexidecimal integer constants (such as `0x01ff`).
+* osl now accepts hexadecimal integer constants (such as `0x01ff`).
   #653 (1.8.1)
 
 Standard library additions/changes:
@@ -1198,7 +1198,7 @@ Standard library additions/changes:
   message passed to the renderer (leaving error handling and reporting
   entirely up to the shader). #686 (1.8.2)
 * `select(a,b,cond)` is similar to `mix`, but without interpolation --
-  if cond is 0, returns a, else return b (per comonent). #696 (1.8.3)
+  if cond is 0, returns a, else return b (per component). #696 (1.8.3)
 
 API changes, new options, new ShadingSystem features (for renderer writers):
 * ShadingSystem API changes:
@@ -1228,7 +1228,7 @@ API changes, new options, new ShadingSystem features (for renderer writers):
     * When the attribute "profile" is set, the statistics report will
       include a count of the total number of noise calls. #642 (1.8.0)
     * ShadingSystem and shader group attribute "exec_repeat" (int; default is
-      1) can artifically manipulate how many times the shader is executed.
+      1) can artificially manipulate how many times the shader is executed.
       This can sometimes be handy for tricky benchmarks. #659 (1.8.2)
     * Attribute "force_derivs" can force derivatives to be computed
       everywhere. (Mostly good for debugging or benchmarking derivatives,
@@ -1241,7 +1241,7 @@ Performance improvements:
   output parameters and what optimizations can be done based on that. #657
   (1.8.1)
 * Better runtime optimization of certain matrix constructors. #676 (1.8.2)
-* Improved alias analyis for parameters with simple initialization ops.
+* Improved alias analysis for parameters with simple initialization ops.
   #679 (1.8.2)
 * Runtime optimizer now constant-folds bitwise `&`, `|`, `^`, `~`
   operators. #682 (1.8.2)
@@ -1362,7 +1362,7 @@ Release 1.7.2 -- 1 Mar 2016 (compared to 1.7.1)
 
 Release 1.7 [1.7.1] -- 29 Jan 2016 (compared to 1.6)
 --------------------------------------------------
-Language, standard libary, and compiler changes (for shader writers):
+Language, standard library, and compiler changes (for shader writers):
 * Language features:
    * Support for closure arrays in shader inputs. #544 (1.7.0)
    * Disallow quote strings from spanning lines (just like you can't in
@@ -1373,7 +1373,7 @@ Language, standard libary, and compiler changes (for shader writers):
    * isconstant(expr) returns 1 if the expression can be turned into
      a constant at runtime (with full knowledge of the shader group's
      parameter values and connections). This is helpful mostly for advanced
-     shader writers to verify their assuptions about which expressions can
+     shader writers to verify their assumptions about which expressions can
      end up being fully constant-folded by the runtime optimizer. #578 (1.7.1)
    * New behavior of isconnected(parmaname): return 1 if the parameter
      is connected to an upstream layer, 2 if connected to a downstream layer,
@@ -1407,7 +1407,7 @@ API changes, new options, new ShadingSystem features (for renderer writers):
      users). (1.7.0)
    * "opt_passes" controls the number of optimization passes over each
      instance (default: 10). This is mainly for developer debugging, you
-     should not need to change or expost this. #576 (1.7.1)
+     should not need to change or expose this. #576 (1.7.1)
    * "connection_error" controls whether a bad connection is considered
      a true error (or is just ignored). #583 (1.7.1)
 * Shader group attribute additions/changes:
@@ -1505,7 +1505,7 @@ Documentation:
 
 Release 1.6 -- 26 May 2015 (compared to 1.5)
 -----------------------------------------------
-Language, standard libary, and compiler changes (for shader writers):
+Language, standard library, and compiler changes (for shader writers):
 * It is now supported to have shader parameters that are arrays of
   unspecified length (like 'float foo[] = {...}'). Their length will be
   fixed when they are assigned an instance value of fixed length, or when
@@ -1646,11 +1646,11 @@ Bug fixes and other improvements:
 * Improved detection of shader groups that optimize away to nothing. (1.6.2)
 * Fix of possible (but never reported) bug with instancing merging.
   #431 (1.6.2)
-* Fix incorrect oslc code generation when assigining an element of an
+* Fix incorrect oslc code generation when assigning an element of an
   array-of-structs to another element of array-of-structs. #432 (1.6.2)
 * Bug fix in code generation of constant folding of getmatrix() calls
   (never reported symptomatic). #440 (1.6.2)
-* Fixes to constuction of temporary filename used for group serialization
+* Fixes to construction of temporary filename used for group serialization
   archive. (1.6.2)
 * Internals fix: logb() incorrectly declaring derivative functions (it
   has no derivs). #437 (1.5.12/1.6.2)
@@ -1666,7 +1666,7 @@ Bug fixes and other improvements:
 * Bug fix: runtime optimizer should not consider a layer unused if its
   outputs are renderer outputs. Also fix subtle bug in detecting whether
   an output param is a renderer output. #457 (1.6.2)
-* Clear derivatives properly in the implementaiton of gettextureinfo.
+* Clear derivatives properly in the implementation of gettextureinfo.
   #463 (1.6.2)
 * Correct handling of point-like shader parameters defined with default
   values that use the constructors that provide "space" names. #465 (1.6.2)

CONTRIBUTING.md

@@ -309,7 +309,7 @@ Namespaces: yes, use them!
 Prefer C++11 `std` rather than Boost or other third party libraries, where
 both can do the same task.
 
-If you see a third party libary already used as a dependency (such as OIIO,
+If you see a third party library already used as a dependency (such as OIIO,
 Boost, Ilmbase, or LLVM), feel free to any of its public features in OSL
 internals (provided those features are present in the minimum version of
 that library that we support).

GOVERNANCE.md

@@ -128,7 +128,7 @@ represents the project at all ASWF TAC meetings.
 (Proposed list, subject to revision, some are placeholders while their
 organizations decide who should be the permanent representative)
 
-* **Chair and TAC repreentative**: Chris Kulla - Sony Pictures Imageworks
+* **Chair and TAC representative**: Chris Kulla - Sony Pictures Imageworks
 * **Chief Architect**: Larry Gritz, Sony Pictures Imageworks
 
 * Luke Emrose - Animal Logic

LICENSE-THIRD-PARTY.md

@@ -12,7 +12,7 @@ In no particular order:
 BSD 3-Clause License
 SPDX-License-Identifier: BSD-3-Clause
 
-* OpenImgageIO Copyright (c) Larry Gritz, et al.
+* OpenImageIO Copyright (c) Larry Gritz, et al.
   https://github.com/OpenImageIO/oiio
 
   Many bits of code flow back and forth between OIIO and OSL, especially

README.md

@@ -386,7 +386,7 @@ Simple "how do I...", "I'm having trouble", or "is this a bug" questions are
 best asked on the [osl-dev developer mail
 list](https://lists.aswf.io/g/osl-dev).
 That's where the most people will see it and potentially be able to answer
-your question quickly (moreso than a GH "issue").
+your question quickly (more so than a GH "issue").
 
 Bugs, build problems, and discovered vulnerabilities that you are relatively
 certain is a legit problem in the code, and for which you can give clear

src/cmake/check_is_enabled.cmake

@@ -5,7 +5,7 @@
 # Is the named package "enabled" via our disabling convention? If either
 # USE_pkgname (or the all-uppercase USE_PKGNAME, or ENABLE_pkgname, or
 # ENABLE_PKGNAME) exists as either a CMake or environment variable, is
-# nonempty by contains a non-true/nonnzero value, store false in the
+# nonempty by contains a non-true/nonzero value, store false in the
 # variable named by <enablevar>, otherwise store true.
 function (check_is_enabled pkgname enablevar)
     string (TOUPPER ${pkgname} pkgname_upper)

src/cmake/compiler.cmake

@@ -116,7 +116,7 @@ if (${CXX_VISIBILITY_PRESET} STREQUAL "hidden" AND
     (CMAKE_SYSTEM_NAME MATCHES "Linux|kFreeBSD" OR CMAKE_SYSTEM_NAME STREQUAL "GNU"))
     # Linux/FreeBSD/Hurd: also hide all the symbols of dependent libraries
     # to prevent clashes if an app using this project is linked against
-    # other verions of our dependencies.
+    # other versions of our dependencies.
     set (VISIBILITY_MAP_COMMAND "-Wl,--version-script=${VISIBILITY_MAP_FILE}")
 endif ()
 
@@ -202,9 +202,9 @@ endif ()
 # Use ccache if found
 #
 # This can really speed up compilation by caching object files that have
-# been compiled previously with idential arguments and inputs. Putting this
+# been compiled previously with identical arguments and inputs. Putting this
 # logic here makes it work even if the user is unaware of ccache. If it's
-# not found on the system, it will simply be silnetly not used.
+# not found on the system, it will simply be silently not used.
 option (USE_CCACHE "Use ccache if found" ON)
 find_program (CCACHE_FOUND ccache)
 if (CCACHE_FOUND AND USE_CCACHE)
@@ -249,8 +249,8 @@ endif ()
 ###########################################################################
 # SIMD and machine architecture options.
 #
-# The USE_SIMD optinon may be set to a comma-separated list of machine /
-# instruction set optinos, such as "avx3,f16c". The list will be parsed and
+# The USE_SIMD option may be set to a comma-separated list of machine /
+# instruction set options, such as "avx3,f16c". The list will be parsed and
 # the proper compiler directives added to generate code for those ISA
 # capabilities.
 #

src/cmake/externalpackages.cmake

@@ -53,7 +53,7 @@ set (OPTIONAL_DEPS "" CACHE STRING
 # following extra features:
 #   * If either `USE_pkgname` or the all-uppercase `USE_PKGNAME` (or
 #     `ENABLE_pkgname` or `ENABLE_PKGNAME`) exists as either a CMake or
-#     environment variable, is nonempty by contains a non-true/nonnzero
+#     environment variable, is nonempty by contains a non-true/nonzero
 #     value, do not search for or use the package. The optional ENABLE <var>
 #     arguments allow you to override the name of the enabling variable. In
 #     other words, support for the dependency is presumed to be ON, unless
@@ -289,7 +289,7 @@ if (USE_CUDA OR USE_OPTIX)
 
     STRING (FIND ${LLVM_TARGETS} "NVPTX" nvptx_index)
     if (NOT ${nvptx_index} GREATER -1)
-        message (FATAL_ERROR "NVTPX target is not available in the provided LLVM build")
+        message (FATAL_ERROR "NVPTX target is not available in the provided LLVM build")
     endif()
 
     if (${CUDA_VERSION} VERSION_GREATER 8 AND ${LLVM_VERSION} VERSION_LESS 6)

src/cmake/fancy_add_executable.cmake

@@ -4,9 +4,9 @@
 
 # Macro to add an executable build target. The executable name can be
 # specified with NAME, otherwise is inferred from the subdirectory name. The
-# source files of the binary can be specifeied with SRC, otherwise are
+# source files of the binary can be specified with SRC, otherwise are
 # inferred to be all the .cpp files within the subdirectory. Optional
-# compile DEFINITIONS, private INCLUDE_DIRS, and priviate LINK_LIBRARIES may
+# compile DEFINITIONS, private INCLUDE_DIRS, and private LINK_LIBRARIES may
 # also be specified.
 #
 # The executable may be disabled individually using any of the usual