fix(fake_test_node, osqp_interface, qp_interface): remove unnecessary cppcheck inline suppressions

veqcc · veqcc · commit 08ec039ed173 · 2024-07-04T23:49:30.000+09:00
Signed-off-by: Ryuta Kambe &lt;ryuta.kambe@tier4.jp&gt;
diff --git a/common/fake_test_node/test/test_fake_test_node.cpp b/common/fake_test_node/test/test_fake_test_node.cpp
@@ -97,7 +97,6 @@ TEST_F(FakeNodeFixture, Test)
 
 INSTANTIATE_TEST_SUITE_P(
   FakeNodeFixtureTests, FakeNodeFixtureParametrized,
-  // cppcheck-suppress syntaxError  // cppcheck doesn't like the trailing comma.
   ::testing::Values(-5, 0, 42));
 
 /// @test Test that we can use a parametrized test.
diff --git a/common/osqp_interface/test/test_osqp_interface.cpp b/common/osqp_interface/test/test_osqp_interface.cpp
@@ -37,7 +37,6 @@ namespace
 // y = [-2.9, 0.0, 0.2, 0.0]`
 // obj = 1.88
 
-// cppcheck-suppress syntaxError
 TEST(TestOsqpInterface, BasicQp)
 {
   using autoware::common::osqp::calCSCMatrix;
diff --git a/common/qp_interface/test/test_osqp_interface.cpp b/common/qp_interface/test/test_osqp_interface.cpp
@@ -37,7 +37,6 @@ namespace
 // y = [-2.9, 0.0, 0.2, 0.0]`
 // obj = 1.88
 
-// cppcheck-suppress syntaxError
 TEST(TestOsqpInterface, BasicQp)
 {
   using qp::calCSCMatrix;
diff --git a/common/qp_interface/test/test_proxqp_interface.cpp b/common/qp_interface/test/test_proxqp_interface.cpp
@@ -38,7 +38,6 @@ namespace
 // y = [-2.9, 0.0, 0.2, 0.0]`
 // obj = 1.88
 
-// cppcheck-suppress syntaxError
 TEST(TestProxqpInterface, BasicQp)
 {
   auto check_result = [](const auto & solution) {

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,6 @@ namespace`
`37`	`37`	// y = [-2.9, 0.0, 0.2, 0.0]`
`38`	`38`	`// obj = 1.88`
`39`	`39`
`40`		`-// cppcheck-suppress syntaxError`
`41`	`40`	`TEST(TestOsqpInterface, BasicQp)`
`42`	`41`	`{`
`43`	`42`	`using autoware::common::osqp::calCSCMatrix;`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,6 @@ namespace`
`38`	`38`	// y = [-2.9, 0.0, 0.2, 0.0]`
`39`	`39`	`// obj = 1.88`
`40`	`40`
`41`		`-// cppcheck-suppress syntaxError`
`42`	`41`	`TEST(TestProxqpInterface, BasicQp)`
`43`	`42`	`{`
`44`	`43`	`auto check_result = [](const auto & solution) {`