Fix typos (danmar#1568)

Author: practicalswift

addons/misra.py

@@ -524,7 +524,7 @@ def generateTable():
 def remove_file_prefix(file_path, prefix):
     """
     Remove a file path prefix from a give path.  leftover
-    directory seperators at the beginning of a file
+    directory separators at the beginning of a file
     after the removal are also stripped.
 
     Example:
@@ -537,7 +537,7 @@ def remove_file_prefix(file_path, prefix):
     result = None
     if file_path.startswith(prefix):
         result = file_path[len(prefix):]
-        # Remove any leftover directory seperators at the
+        # Remove any leftover directory separators at the
         # beginning
         result = result.lstrip('\\/')
     else:
@@ -565,7 +565,7 @@ def __init__(self):
         # Dictionary of dictionaries for rules to suppress
         # Dict1 is keyed by rule number in the hundreds format of
         # Major *  100 + minor. ie Rule 5.2 = (5*100) + 2
-        # Dict 2 is keyed by filename.  An entry of None means suppress globaly.
+        # Dict 2 is keyed by filename.  An entry of None means suppress globally.
         # Each file name entry contails a list of tuples of (lineNumber, symbolName)
         # or an item of None which indicates suppress rule for the entire file.
         # The line and symbol name tuple may have None as either of its elements but
@@ -1882,7 +1882,7 @@ def showSuppressedRules(self):
     def setFilePrefix(self, prefix):
         """
         Set the file prefix to ignnore from files when matching
-        supression files
+        suppression files
         """
         self.filePrefix = prefix
 
@@ -2179,7 +2179,7 @@ def parseDump(self, dumpfile):
                         print('Not expected: ' + actual)
                         exitCode = 1
 
-                # Exisitng behavior of verify mode is to exit
+                # Existing behavior of verify mode is to exit
                 # on the first un-expected output.
                 # TODO: Is this required? or can it be moved to after
                 # all input files have been processed

addons/namingng.py

@@ -19,7 +19,7 @@
 #                          "uint32_t": "ui32"}
 # }
 #
-# RE_VARNAME, RE_PRIVATE_MEMBER_VARIABE and RE_FUNCTIONNAME are regular expressions to cover hte basic names
+# RE_VARNAME, RE_PRIVATE_MEMBER_VARIABE and RE_FUNCTIONNAME are regular expressions to cover the basic names
 # In var_prefixes and function_prefixes there are the variable-type/prefix pairs
 
 import cppcheckdata

cfg/zlib.cfg

@@ -1,6 +1,6 @@
 <?xml version="1.0"?>
 <def format="2">
-  <!-- Reference to zlib documention https://www.zlib.net/manual.html -->
+  <!-- Reference to zlib documentation https://www.zlib.net/manual.html -->
   <!-- Data structures and types -->
   <podtype name="alloc_func"/>
   <podtype name="Byte" sign="u" size="1"/>

cmake/options.cmake

@@ -23,7 +23,7 @@ option(ANALYZE_DATAFLOW     "Clang dynamic analyzer: general dynamic dataflow an
 option(WARNINGS_ARE_ERRORS  "Treat warnings as errors"                                      OFF)
 option(WARNINGS_ANSI_ISO    "Issue all the mandatory diagnostics Listed in C standard"      ON)
 
-set(USE_MATCHCOMPILER "Auto" CACHE STRING "Usage of match compliler")
+set(USE_MATCHCOMPILER "Auto" CACHE STRING "Usage of match compiler")
 set_property(CACHE USE_MATCHCOMPILER PROPERTY STRINGS Auto Off On Verify) 
 if (USE_MATCHCOMPILER STREQUAL "Auto")
     if (NOT CMAKE_BUILD_TYPE STREQUAL "Debug")

doxyfile

@@ -840,7 +840,7 @@ USE_HTAGS              = NO
 VERBATIM_HEADERS       = YES
 
 # If CLANG_ASSISTED_PARSING is set to YES, then doxygen will use the clang parser
-# for more acurate parsing at the cost of reduced performance. This can be
+# for more accurate parsing at the cost of reduced performance. This can be
 # particularly helpful with template rich C++ code for which doxygen's built-in
 # parser lacks the necessary type information.
 
@@ -929,7 +929,7 @@ HTML_STYLESHEET        =
 # user-defined cascading style sheet that is included after the standard
 # style sheets created by doxygen. Using this option one can overrule
 # certain style aspects. This is preferred over using HTML_STYLESHEET
-# since it does not replace the standard style sheet and is therefor more
+# since it does not replace the standard style sheet and is therefore more
 # robust against future updates. Doxygen will copy the style sheet file to
 # the output directory.
 

lib/valueflow.cpp

@@ -1971,7 +1971,7 @@ static bool valueFlowForward(Token * const               startToken,
             Token *lambdaEndToken = const_cast<Token *>(findLambdaEndToken(tok2));
             if (isVariableChanged(lambdaEndToken->link(), lambdaEndToken, varid, var->isGlobal(), settings, tokenlist->isCPP()))
                 return false;
-            // Dont skip lambdas for lifetime values
+            // Don't skip lambdas for lifetime values
             if (!std::all_of(values.begin(), values.end(), std::mem_fn(&ValueFlow::Value::isLifetimeValue))) {
                 tok2 = lambdaEndToken;
                 continue;
@@ -2682,7 +2682,7 @@ struct LifetimeStore {
         value.tokvalue = var->nameToken();
         value.errorPath = errorPath;
         value.lifetimeKind = type;
-        // Dont add the value a second time
+        // Don't add the value a second time
         if (std::find(tok->values().begin(), tok->values().end(), value) != tok->values().end())
             return;
         setTokenValue(tok, value, tokenlist->getSettings());
@@ -2714,7 +2714,7 @@ struct LifetimeStore {
             value.tokvalue = var->nameToken();
             value.errorPath = errorPath;
             value.lifetimeKind = type;
-            // Dont add the value a second time
+            // Don't add the value a second time
             if (std::find(tok->values().begin(), tok->values().end(), value) != tok->values().end())
                 continue;
             setTokenValue(tok, value, tokenlist->getSettings());
@@ -4190,7 +4190,7 @@ static void valueFlowSubFunction(TokenList *tokenlist, const Settings *settings)
             // passing value(s) to function
             std::list<ValueFlow::Value> argvalues(getFunctionArgumentValues(argtok));
 
-            // Dont forward lifetime values
+            // Don't forward lifetime values
             argvalues.remove_if(std::mem_fn(&ValueFlow::Value::isLifetimeValue));
 
             if (argvalues.empty())

test/testautovariables.cpp

@@ -1571,38 +1571,38 @@ class TestAutoVariables : public TestFixture {
               "}\n");
         ASSERT_EQUALS("", errout.str());
 
-        // Dont decay std::array
+        // Don't decay std::array
         check("std::array<char, 1> f() {\n"
               "    std::array<char, 1> x;\n"
               "    return x;\n"
               "}\n");
         ASSERT_EQUALS("", errout.str());
 
-        // Make sure we dont hang
+        // Make sure we don't hang
         check("struct A;\n"
               "void f() {\n"
               "    using T = A[3];\n"
               "    A &&a = T{1, 2, 3}[1];\n"
               "}\n");
         ASSERT_EQUALS("", errout.str());
 
-        // Make sure we dont hang
+        // Make sure we don't hang
         check("struct A;\n"
               "void f() {\n"
               "    using T = A[3];\n"
               "    A &&a = T{1, 2, 3}[1]();\n"
               "}\n");
         ASSERT_EQUALS("", errout.str());
 
-        // Make sure we dont hang
+        // Make sure we don't hang
         check("struct A;\n"
               "void f() {\n"
               "    using T = A[3];\n"
               "    A &&a = T{1, 2, 3}[1];\n"
               "}\n");
         ASSERT_EQUALS("", errout.str());
 
-        // Make sure we dont hang
+        // Make sure we don't hang
         check("struct A;\n"
               "void f() {\n"
               "    using T = A[3];\n"

test/testcondition.cpp

@@ -2158,7 +2158,7 @@ class TestCondition : public TestFixture {
               "  X(do);\n"
               "  if (x > 100) {}\n"
               "}");
-        // TODO: we should probably throw unknownMacro InternalError. Complain that the macro X must be defined. We cant check the code well without the definition.
+        // TODO: we should probably throw unknownMacro InternalError. Complain that the macro X must be defined. We can't check the code well without the definition.
         TODO_ASSERT_EQUALS("[test.cpp:2] -> [test.cpp:4]: (style) Condition 'x>100' is always false\n", "", errout.str());
 
         check("void f(const int *i) {\n"

test/testconstructors.cpp

@@ -3484,12 +3484,12 @@ class TestConstructors : public TestFixture {
     }
 
     void templateConstructor() { // ticket #7942
-        check("template <class T> struct Containter {\n"
-              "  Containter();\n"
+        check("template <class T> struct Container {\n"
+              "  Container();\n"
               "  T* mElements;\n"
               "};\n"
-              "template <class T> Containter<T>::Containter() : mElements(nullptr) {}\n"
-              "Containter<int> intContainer;");
+              "template <class T> Container<T>::Container() : mElements(nullptr) {}\n"
+              "Container<int> intContainer;");
         ASSERT_EQUALS("", errout.str());
     }
 

test/testgarbage.cpp

@@ -824,7 +824,7 @@ class TestGarbage : public TestFixture {
 
     void garbageCode98() { // #6838
         ASSERT_THROW(checkCode("for (cocon To::ta@Taaaaaforconst oken aaaaaaaaaaaa5Dl()\n"
-                               "const unsiged in;\n"
+                               "const unsigned in;\n"
                                "fon *tok = f);.s(Token i = d-)L;"), InternalError);
     }