C++: Updated autoformat.

geoffw0 · geoffw0 · commit fddd35315528 · 2020-11-20T20:15:45.000Z
diff --git a/cpp/ql/src/semmle/code/cpp/models/implementations/Iterator.qll b/cpp/ql/src/semmle/code/cpp/models/implementations/Iterator.qll
@@ -308,8 +308,10 @@ private class IteratorAssignmentMemberOperator extends MemberFunction, TaintFunc
 private class BeginOrEndFunction extends MemberFunction, TaintFunction, GetIteratorFunction {
   BeginOrEndFunction() {
     this
-        .hasName(["begin", "cbegin", "rbegin", "crbegin", "end", "cend", "rend", "crend",
-              "before_begin", "cbefore_begin"]) and
+        .hasName([
+            "begin", "cbegin", "rbegin", "crbegin", "end", "cend", "rend", "crend", "before_begin",
+            "cbefore_begin"
+          ]) and
     this.getType().getUnspecifiedType() instanceof Iterator
   }
 
diff --git a/cpp/ql/src/semmle/code/cpp/models/implementations/Pure.qll b/cpp/ql/src/semmle/code/cpp/models/implementations/Pure.qll
@@ -6,9 +6,11 @@ import semmle.code.cpp.models.interfaces.SideEffect
 private class PureStrFunction extends AliasFunction, ArrayFunction, TaintFunction,
   SideEffectFunction {
   PureStrFunction() {
-    hasGlobalOrStdName(["atof", "atoi", "atol", "atoll", "strcasestr", "strchnul", "strchr",
-          "strchrnul", "strstr", "strpbrk", "strcmp", "strcspn", "strncmp", "strrchr", "strspn",
-          "strtod", "strtof", "strtol", "strtoll", "strtoq", "strtoul"])
+    hasGlobalOrStdName([
+        "atof", "atoi", "atol", "atoll", "strcasestr", "strchnul", "strchr", "strchrnul", "strstr",
+        "strpbrk", "strcmp", "strcspn", "strncmp", "strrchr", "strspn", "strtod", "strtof",
+        "strtol", "strtoll", "strtoq", "strtoul"
+      ])
   }
 
   override predicate hasArrayInput(int bufParam) {
diff --git a/cpp/ql/src/semmle/code/cpp/models/implementations/Strcpy.qll b/cpp/ql/src/semmle/code/cpp/models/implementations/Strcpy.qll
@@ -14,20 +14,24 @@ import semmle.code.cpp.models.interfaces.SideEffect
 class StrcpyFunction extends ArrayFunction, DataFlowFunction, TaintFunction, SideEffectFunction {
   StrcpyFunction() {
     getName() =
-      ["strcpy", // strcpy(dst, src)
-          "wcscpy", // wcscpy(dst, src)
-          "_mbscpy", // _mbscpy(dst, src)
-          "strncpy", // strncpy(dst, src, max_amount)
-          "_strncpy_l", // _strncpy_l(dst, src, max_amount, locale)
-          "wcsncpy", // wcsncpy(dst, src, max_amount)
-          "_wcsncpy_l", // _wcsncpy_l(dst, src, max_amount, locale)
-          "_mbsncpy", // _mbsncpy(dst, src, max_amount)
-          "_mbsncpy_l"] // _mbsncpy_l(dst, src, max_amount, locale)
+      [
+        "strcpy", // strcpy(dst, src)
+        "wcscpy", // wcscpy(dst, src)
+        "_mbscpy", // _mbscpy(dst, src)
+        "strncpy", // strncpy(dst, src, max_amount)
+        "_strncpy_l", // _strncpy_l(dst, src, max_amount, locale)
+        "wcsncpy", // wcsncpy(dst, src, max_amount)
+        "_wcsncpy_l", // _wcsncpy_l(dst, src, max_amount, locale)
+        "_mbsncpy", // _mbsncpy(dst, src, max_amount)
+        "_mbsncpy_l"
+      ] // _mbsncpy_l(dst, src, max_amount, locale)
     or
     getName() =
-      ["strcpy_s", // strcpy_s(dst, max_amount, src)
-          "wcscpy_s", // wcscpy_s(dst, max_amount, src)
-          "_mbscpy_s"] and // _mbscpy_s(dst, max_amount, src)
+      [
+        "strcpy_s", // strcpy_s(dst, max_amount, src)
+        "wcscpy_s", // wcscpy_s(dst, max_amount, src)
+        "_mbscpy_s"
+      ] and // _mbscpy_s(dst, max_amount, src)
     // exclude the 2-parameter template versions
     // that find the size of a fixed size destination buffer.
     getNumberOfParameters() = 3
