clang tidy

Nettozx · Nettozx · commit 68cc2148e941 · 2025-09-29T23:38:07.000-07:00
diff --git a/test/testsarifreport.cpp b/test/testsarifreport.cpp
@@ -60,7 +60,7 @@ class TestSarifReport : public TestFixture
     }
 
     // Helper to create an ErrorMessage
-    ErrorMessage createErrorMessage(const std::string& id,
+    static ErrorMessage createErrorMessage(const std::string& id,
                                     Severity severity,
                                     const std::string& msg,
                                     const std::string& file = "test.cpp",
@@ -79,7 +79,7 @@ class TestSarifReport : public TestFixture
     }
 
     // Helper to parse JSON and validate structure
-    bool parseAndValidateJson(const std::string& json, picojson::value& root)
+    static bool parseAndValidateJson(const std::string& json, picojson::value& root)
     {
         std::string parseError = picojson::parse(root, json);
         return parseError.empty() && root.is<picojson::object>();

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ class TestSarifReport : public TestFixture`
`60`	`60`	`}`
`61`	`61`
`62`	`62`	`// Helper to create an ErrorMessage`
`63`		`- ErrorMessage createErrorMessage(const std::string& id,`
	`63`	`+ static ErrorMessage createErrorMessage(const std::string& id,`
`64`	`64`	`Severity severity,`
`65`	`65`	`const std::string& msg,`
`66`	`66`	`const std::string& file = "test.cpp",`
`@@ -79,7 +79,7 @@ class TestSarifReport : public TestFixture`
`79`	`79`	`}`
`80`	`80`
`81`	`81`	`// Helper to parse JSON and validate structure`
`82`		`- bool parseAndValidateJson(const std::string& json, picojson::value& root)`
	`82`	`+ static bool parseAndValidateJson(const std::string& json, picojson::value& root)`
`83`	`83`	`{`
`84`	`84`	`std::string parseError = picojson::parse(root, json);`
`85`	`85`	`return parseError.empty() && root.is<picojson::object>();`