Fix typos in repo

Author: mcbarton

CMakeLists.txt

@@ -440,8 +440,8 @@ endif()
 
 # Generate docs for CppInterOp
 option(CPPINTEROP_INCLUDE_DOCS "Generate build targets for the CppInterOp docs.")
-option(CPPINTEROP_ENABLE_DOXYGEN "Use doxygen to generate CppInterOp interal API documentation.")
-option(CPPINTEROP_ENABLE_SPHINX "Use sphinx to generage CppInterOp user documentation")
+option(CPPINTEROP_ENABLE_DOXYGEN "Use doxygen to generate CppInterOp internal API documentation.")
+option(CPPINTEROP_ENABLE_SPHINX "Use sphinx to generate CppInterOp user documentation")
 
 
 if(MSVC)

CONTRIBUTING.md

@@ -42,7 +42,7 @@ projects such as CppInterOp. We have our documentation in the repository which i
 rendered in the [CppInterOp.readthedocs](https://cppinterop.readthedocs.io/en/latest/) website.
 Documentation modifications happen by proposing a pull request.
 
-## Creating a successfull pull request
+## Creating a successful pull request
 
 To propose a code modification we use the pull requests. Pull requests which
 review quickly and successfully share several common traits:
@@ -71,4 +71,4 @@ review quickly and successfully share several common traits:
 ### Developer Documentation  
 
 We have documented several useful hints that usually help when addressing issues
-as they come during developement time in our [developer documentation](https://cppinterop.readthedocs.io/en/latest/InstallationAndUsage.html).  
+as they come during development time in our [developer documentation](https://cppinterop.readthedocs.io/en/latest/InstallationAndUsage.html).  

cmake/modules/GoogleTest.cmake

@@ -81,7 +81,7 @@ else()
 endif()
 
 # Use gmock_main instead of gtest_main because it initializes gtest as well.
-# Note: The libraries are listed in reverse order of their dependancies.
+# Note: The libraries are listed in reverse order of their dependencies.
 foreach(lib gtest gtest_main gmock gmock_main)
   add_library(${lib} IMPORTED STATIC GLOBAL)
   set_target_properties(${lib} PROPERTIES

docs/doxygen.cfg.in

@@ -70,7 +70,7 @@ CREATE_SUBDIRS         = NO
 # Croatian, Czech, Danish, Dutch, Esperanto, Farsi, Finnish, French, German, 
 # Greek, Hungarian, Italian, Japanese, Japanese-en (Japanese with English 
 # messages), Korean, Korean-en, Lithuanian, Norwegian, Macedonian, Persian, 
-# Polish, Portuguese, Romanian, Russian, Serbian, Serbian-Cyrilic, Slovak, 
+# Polish, Portuguese, Romanian, Russian, Serbian, Serbian-Cyrillic, Slovak, 
 # Slovene, Spanish, Swedish, Ukrainian, and Vietnamese.
 
 OUTPUT_LANGUAGE        = English
@@ -1185,7 +1185,7 @@ EXT_LINKS_IN_WINDOW    = NO
 
 FORMULA_FONTSIZE       = 10
 
-# Use the FORMULA_TRANPARENT tag to determine whether or not the images
+# Use the FORMULA_TRANSPARENT tag to determine whether or not the images
 # generated for formulas are transparent PNGs. Transparent PNGs are not
 # supported properly for IE 6.0, but are supported on all modern browsers.
 #

docs/doxygen.css

@@ -313,7 +313,7 @@ HR { height: 1px;
 
 /* 
  * LLVM Modifications.
- * Note: Everything above here is generated with "doxygen -w htlm" command. See
+ * Note: Everything above here is generated with "doxygen -w html" command. See
  * "doxygen --help" for details. What follows are CSS overrides for LLVM 
  * specific formatting. We want to keep the above so it can be replaced with
  * subsequent doxygen upgrades.

include/clang/Interpreter/CppInterOp.h

@@ -730,7 +730,7 @@ namespace Cpp {
   ///
   ///\param[in] candidates - vector of overloads that come under the
   ///           parent scope and have the same name
-  ///\param[in] explicit_types - set of expicitly instantiated template types
+  ///\param[in] explicit_types - set of explicitly instantiated template types
   ///\param[in] arg_types - set of argument types
   ///\returns Instantiated function pointer
   CPPINTEROP_API TCppFunction_t
@@ -791,7 +791,7 @@ namespace Cpp {
   /// Append all Code completion suggestions to Results.
   ///\param[out] Results - CC suggestions for code fragment. Suggestions are
   /// appended.
-  ///\param[in] code - code fragmet to complete
+  ///\param[in] code - code fragment to complete
   ///\param[in] complete_line - position (line) in code for suggestion
   ///\param[in] complete_column - position (column) in code for suggestion
   CPPINTEROP_API void CodeComplete(std::vector<std::string>& Results,

lib/Interpreter/Compatibility.h

@@ -483,7 +483,7 @@ inline std::string MakeResourceDir(llvm::StringRef Dir) {
   return std::string(P.str());
 }
 
-// Clang >= 16 (=16 with Value patch) change castAs to converTo
+// Clang >= 16 (=16 with Value patch) change castAs to convertTo
 #ifdef CPPINTEROP_USE_CLING
 template <typename T> inline T convertTo(cling::Value V) {
   return V.castAs<T>();

lib/Interpreter/CppInterOp.cpp

@@ -1421,7 +1421,7 @@ namespace Cpp {
         // not C. That means BaseCXXRD derives from C. Offset needs to be
         // calculated for Derived class
 
-        // Depth first Search is performed to the class that declears FD from
+        // Depth first Search is performed to the class that declares FD from
         // the base class
         std::vector<CXXRecordDecl*> stack;
         std::map<CXXRecordDecl*, CXXRecordDecl*> direction;
@@ -3647,7 +3647,7 @@ namespace Cpp {
       m_DupFD = dup(FD);
 
       // Flush now or can drop the buffer when dup2 is called with Fd later.
-      // This seems only neccessary when piping stdout or stderr, but do it
+      // This seems only necessary when piping stdout or stderr, but do it
       // for ttys to avoid over complicated code for minimal benefit.
       ::fflush(FD == STDOUT_FILENO ? stdout : stderr);
       if (dup2(fileno(m_TempFile.get()), FD) < 0)

lib/Interpreter/CppInterOpInterpreter.h

@@ -357,7 +357,7 @@ class Interpreter {
         const_cast<const Interpreter*>(this)->getDynamicLibraryManager());
   }
 
-  ///\brief Adds multiple include paths separated by a delimter.
+  ///\brief Adds multiple include paths separated by a delimiter.
   ///
   ///\param[in] PathsStr - Path(s)
   ///\param[in] Delim - Delimiter to separate paths or NULL if a single path

lib/Interpreter/DynamicLibraryManager.cpp

@@ -51,11 +51,12 @@ namespace Cpp {
     };
 
     // Behaviour is to not add paths that don't exist...In an interpreted env
-    // does this make sense? Path could pop into existance at any time.
+    // does this make sense? Path could pop into existence at any time.
     for (const char* Var : kSysLibraryEnv) {
       if (const char* Env = GetEnv(Var)) {
         SmallVector<StringRef, 10> CurPaths;
-        SplitPaths(Env, CurPaths, utils::kPruneNonExistant, Cpp::utils::platform::kEnvDelim);
+        SplitPaths(Env, CurPaths, utils::kPruneNonExistent,
+                   Cpp::utils::platform::kEnvDelim);
         for (const auto& Path : CurPaths)
           addSearchPath(Path);
       }
@@ -86,7 +87,7 @@ namespace Cpp {
                               StringRef libLoader) {
 
     // Handle substitutions (MacOS):
-    // @rpath - This function does not substitute @rpath, becouse
+    // @rpath - This function does not substitute @rpath, because
     //          this variable is already handled by lookupLibrary where
     //          @rpath is replaced with all paths from RPATH one by one.
     // @executable_path - Main program path.