Merge branch 'main' into m0-1-4/issue-6/constexpr-as-use

Author: lcartey

.github/workflows/bump-version.yml

@@ -10,7 +10,7 @@ on:
 jobs:
 
   apply-version-bump:
-    runs-on: ubuntu-latest 
+    runs-on: ubuntu-22.04
     name: Apply Version Bump
     steps:
         - name: Checkout 

.github/workflows/code-scanning-pack-gen.yml

@@ -19,7 +19,7 @@ env:
 jobs:
   prepare-code-scanning-pack-matrix:
     name: Prepare CodeQL Code Scanning pack matrix
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     outputs:
       matrix: ${{ steps.export-code-scanning-pack-matrix.outputs.matrix }}
     steps:

.github/workflows/codeql_unit_tests.yml

@@ -14,7 +14,7 @@ on:
 jobs:
   prepare-unit-test-matrix:
     name: Prepare CodeQL unit test matrix
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     outputs:
       matrix: ${{ steps.export-unit-test-matrix.outputs.matrix }}
     steps:
@@ -157,7 +157,7 @@ jobs:
   validate-test-results:
     name: Validate test results
     needs: [run-test-suites]
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     steps:
       - name: Collect test results
         uses: actions/download-artifact@v2

.github/workflows/create-draft-release.yml

@@ -21,7 +21,7 @@ on:
 jobs:
   create-draft-release:
     name: Create draft release
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     env:      
       # AWS CONFIGURATION
       AWS_EC2_INSTANCE_TYPE: ${{ github.event.inputs.aws_ec2_instance_type }}

.github/workflows/generate-html-docs.yml

@@ -15,7 +15,7 @@ on:
 jobs:
   generate-html-doc:
     name: Generate HTML documentation
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     steps:
       - name: Checkout
         uses: actions/checkout@v2

.github/workflows/standard_library_upgrade_tests.yml

@@ -14,7 +14,7 @@ on:
 jobs:
   prepare-unit-test-matrix:
     name: Prepare CodeQL unit test matrix
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     outputs:
       matrix: ${{ steps.export-unit-test-matrix.outputs.matrix }}
     steps:
@@ -154,7 +154,7 @@ jobs:
   validate-test-results:
     name: Validate test results
     needs: [run-test-suites]
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     steps:
       - name: Install Python
         uses: actions/setup-python@v4

.github/workflows/upgrade_codeql_dependencies.yml

@@ -20,7 +20,7 @@ jobs:
     env:
       CODEQL_CLI_VERSION: ${{ github.event.inputs.codeql_cli_version }}
       CODEQL_LIB_COMMIT: ${{ github.event.inputs.codeql_standard_library_commit }}
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     steps:
       - name: Checkout
         uses: actions/checkout@v2

.github/workflows/validate-coding-standards.yml

@@ -18,7 +18,7 @@ env:
 jobs:
   validate-package-files:
     name: Validate Package Files
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     steps:
       - name: Checkout
         uses: actions/checkout@v2
@@ -63,7 +63,7 @@ jobs:
 
   validate-codeql-format:
     name: "Validate CodeQL Format"
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     steps:
       - name: Checkout
         uses: actions/checkout@v2
@@ -94,7 +94,7 @@ jobs:
 
   validate-query-help-files:
     name: Validate Query Help Files
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     steps:
       - name: Checkout
         uses: actions/checkout@v2
@@ -129,7 +129,7 @@ jobs:
 
   validate-cpp-test-files:
     name: Validate C++ Test Files
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     steps:
       - name: Checkout
         uses: actions/checkout@v2
@@ -152,7 +152,7 @@ jobs:
 
   validate-c-test-files:
     name: Validate C Test Files
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     steps:
       - name: Checkout
         uses: actions/checkout@v2

c/cert/src/qlpack.yml

@@ -1,4 +1,4 @@
 name: cert-c-coding-standards
-version: 2.9.0
+version: 2.11.0-dev
 suites: codeql-suites
 libraryPathDependencies: common-c-coding-standards

c/cert/test/qlpack.yml

@@ -1,4 +1,4 @@
 name: cert-c-coding-standards-tests
-version: 2.9.0
+version: 2.11.0-dev
 libraryPathDependencies: cert-c-coding-standards
 extractor: cpp

c/common/src/qlpack.yml

@@ -1,3 +1,3 @@
 name: common-c-coding-standards
-version: 2.9.0
+version: 2.11.0-dev
 libraryPathDependencies: common-cpp-coding-standards

c/common/test/qlpack.yml

@@ -1,4 +1,4 @@
 name: common-c-coding-standards-tests
-version: 2.9.0
+version: 2.11.0-dev
 libraryPathDependencies: common-c-coding-standards
 extractor: cpp

c/misra/src/qlpack.yml

@@ -1,4 +1,4 @@
 name: misra-c-coding-standards
-version: 2.9.0
+version: 2.11.0-dev
 suites: codeql-suites
 libraryPathDependencies: common-c-coding-standards

c/misra/test/qlpack.yml

@@ -1,4 +1,4 @@
 name: misra-c-coding-standards-tests
-version: 2.9.0
+version: 2.11.0-dev
 libraryPathDependencies: misra-c-coding-standards
 extractor: cpp

cpp/autosar/src/qlpack.yml

@@ -1,4 +1,4 @@
 name: autosar-cpp-coding-standards
-version: 2.9.0
+version: 2.11.0-dev
 suites: codeql-suites
 libraryPathDependencies: common-cpp-coding-standards

cpp/autosar/test/qlpack.yml

@@ -1,4 +1,4 @@
 name: autosar-cpp-coding-standards-tests
-version: 2.9.0
+version: 2.11.0-dev
 libraryPathDependencies: autosar-cpp-coding-standards
 extractor: cpp

cpp/autosar/test/rules/A5-1-9/test.cpp

@@ -86,16 +86,16 @@ void test_noncompliant() {
 #include <string>
 class Test_issue468 {
 public:
-  template <typename... As> static void LogInfo(const As &... rest) {
+  template <typename... As> static void LogInfo(const As &...rest) {
     [](const std::string &s) -> void { LogInfo(s); }; // COMPLIANT
   }
-  template <typename... As> static void LogWarn(const As &... rest) {
+  template <typename... As> static void LogWarn(const As &...rest) {
     [](const std::string &s) -> void { LogWarn(s); }; // COMPLIANT
   }
-  template <typename... As> static void LogError(const As &... rest) {
+  template <typename... As> static void LogError(const As &...rest) {
     [](const std::string &s) -> void { LogError(s); }; // NON_COMPLIANT
   }
-  template <typename... As> static void LogFatal(const As &... rest) {
+  template <typename... As> static void LogFatal(const As &...rest) {
     [](const std::string &s) -> void { LogError(s); }; // NON_COMPLIANT
   }
   void instantiate() {

cpp/autosar/test/rules/A5-2-2/test.cpp

@@ -45,12 +45,12 @@ void test_cpp_style_cast() {
 class A5_2_2a {
 public:
   template <typename... As>
-  static void Foo(const std::string &name, As &&... rest) {
+  static void Foo(const std::string &name, As &&...rest) {
     Fun(Log(
         std::forward<As>(rest)...)); // COMPLIANT - reported as a false positive
   }
 
-  template <typename... As> static std::string Log(As &&... tail) {
+  template <typename... As> static std::string Log(As &&...tail) {
     return std::string();
   }
 

cpp/autosar/test/rules/A7-2-2/EnumerationUnderlyingBaseTypeNotExplicitlyDefined.expected

@@ -1,2 +1,2 @@
 | test.cpp:1:6:1:6 | C | Base type of enumeration is not explicitly specified. |
-| test.cpp:3:12:3:12 | A | Base type of enumeration is not explicitly specified. |
+| test.cpp:5:12:5:12 | A | Base type of enumeration is not explicitly specified. |

cpp/autosar/test/rules/A7-2-2/test.cpp

@@ -1,8 +1,16 @@
 enum C // NON_COMPLIANT
-{ c1 };
+{
+  c1
+};
 enum class A // NON_COMPLIANT
-{ a1 };
+{
+  a1
+};
 enum class B : int // COMPLIANT
-{ b1 };
+{
+  b1
+};
 enum D : int // COMPLIANT
-{ d1 };
+{
+  d1
+};

cpp/autosar/test/rules/A7-2-3/EnumerationsNotDeclaredAsScopedEnumClasses.expected

@@ -1,2 +1,2 @@
 | test.cpp:1:6:1:6 | C | Enum C is not a scoped enum. |
-| test.cpp:11:6:11:6 | D | Enum D is not a scoped enum. |
+| test.cpp:21:6:21:6 | D | Enum D is not a scoped enum. |

cpp/autosar/test/rules/A7-2-3/test.cpp

@@ -1,12 +1,24 @@
 enum C // NON_COMPLIANT
-{ c1 };
+{
+  c1
+};
 enum class A // COMPLIANT - Violates a different rule (A7-2-2)
-{ a1 };
+{
+  a1
+};
 enum struct A1 // COMPLIANT - Violates a different rule (A7-2-2)
-{ a12 };
+{
+  a12
+};
 enum class B : int // COMPLIANT
-{ b1 };
+{
+  b1
+};
 enum struct B1 : int // COMPLIANT
-{ b12 };
+{
+  b12
+};
 enum D : int // NON_COMPLIANT
-{ d1 };
+{
+  d1
+};

cpp/autosar/test/rules/A7-2-4/NoneFirstOrAllEnumeratorsNotInitialized.expected

@@ -1,4 +1,4 @@
-| test.cpp:9:12:9:13 | A2 | Neither none, the first, or all enumerated constants are initialized in this enumeration. |
-| test.cpp:33:14:33:15 | B3 | Neither none, the first, or all enumerated constants are initialized in this enumeration. |
-| test.cpp:45:6:45:6 | D | Neither none, the first, or all enumerated constants are initialized in this enumeration. |
-| test.cpp:53:6:53:7 | E1 | Neither none, the first, or all enumerated constants are initialized in this enumeration. |
+| test.cpp:13:12:13:13 | A2 | Neither none, the first, or all enumerated constants are initialized in this enumeration. |
+| test.cpp:47:14:47:15 | B3 | Neither none, the first, or all enumerated constants are initialized in this enumeration. |
+| test.cpp:63:6:63:6 | D | Neither none, the first, or all enumerated constants are initialized in this enumeration. |
+| test.cpp:75:6:75:7 | E1 | Neither none, the first, or all enumerated constants are initialized in this enumeration. |

cpp/autosar/test/rules/A7-2-4/test.cpp

@@ -1,55 +1,79 @@
 enum class A // COMPLIANT
-{ a1 };
+{
+  a1
+};
 
 enum class A1 // COMPLIANT
-{ a1 = 0,
+{
+  a1 = 0,
   a2 = 1,
-  a3 = 2 };
+  a3 = 2
+};
 
 enum class A2 // NON_COMPLIANT
-{ a1,
+{
+  a1,
   a2 = 0,
-  a3 };
+  a3
+};
 
 enum class A3 // COMPLIANT
-{ a1 = 0,
+{
+  a1 = 0,
   a2,
-  a3 };
+  a3
+};
 
 enum class B : int // COMPLIANT
-{ b1 };
+{
+  b1
+};
 
 enum class B1 : int // COMPLIANT
-{ b1,
+{
+  b1,
   b2,
-  b3 };
+  b3
+};
 
 enum class B2 : int // COMPLIANT
-{ b1 = 0,
+{
+  b1 = 0,
   b2,
-  b3 };
+  b3
+};
 
 int f() {
   enum class B3 : int // NON_COMPLIANT
-  { b1,
+  {
+    b1,
     b2 = 0,
-    b3 = 0 };
+    b3 = 0
+  };
 
   return 0;
 }
 
 enum C // COMPLIANT
-{ c1,
-  c2 };
+{
+  c1,
+  c2
+};
 
 enum D // NON_COMPLIANT
-{ d1,
-  d2 = 0 };
+{
+  d1,
+  d2 = 0
+};
 
 enum E : int // COMPLIANT
-{ e1,
-  e2 };
+{
+  e1,
+  e2
+};
 
 enum E1 : int // NON_COMPLIANT
-{ e11,
-  e22 = 0 };
+{
+  e11,
+  e22 = 0
+};

cpp/autosar/test/rules/A8-4-5/test.cpp

@@ -48,7 +48,7 @@ class TestClass4 {
 
 // template type
 template <typename T, typename... X>
-T test_forward(X &&... f) {        // forward parameter
+T test_forward(X &&...f) {         // forward parameter
   return T{std::forward<X>(f)...}; // COMPLIANT
 }