Skip to content

Commit

Permalink
fix(fake_test_node, osqp_interface, qp_interface): remove unnecessary…
Browse files Browse the repository at this point in the history
… cppcheck inline suppressions (autowarefoundation#7855)

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

Signed-off-by: Ryuta Kambe <[email protected]>

* style(pre-commit): autofix

---------

Signed-off-by: Ryuta Kambe <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
  • Loading branch information
veqcc and pre-commit-ci[bot] authored Jul 5, 2024
1 parent 22daaeb commit 3627afc
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 6 deletions.
4 changes: 1 addition & 3 deletions common/fake_test_node/test/test_fake_test_node.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -96,9 +96,7 @@ 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));
FakeNodeFixtureTests, FakeNodeFixtureParametrized, ::testing::Values(-5, 0, 42));

/// @test Test that we can use a parametrized test.
TEST_P(FakeNodeFixtureParametrized, Test)
Expand Down
1 change: 0 additions & 1 deletion common/osqp_interface/test/test_osqp_interface.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
1 change: 0 additions & 1 deletion common/qp_interface/test/test_osqp_interface.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
1 change: 0 additions & 1 deletion common/qp_interface/test/test_proxqp_interface.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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) {
Expand Down

0 comments on commit 3627afc

Please sign in to comment.