Fix comma problem with INSTANTIATE_TEST_CASE_P

There exists a compiler warning named "-Wgnu-zero-variadic-macro-arguments". If build with gcc this warning does not appear, however, build with clang the compiler warning appears. To avoid this problem a comma is added which resolves the problem.
PilzDE · Nov 11, 2019 · b78c128 · b78c128
1 parent ff178fb
commit b78c128
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/prbt_hardware_support/test/unittests/unittest_operation_mode_setup_executor.cpp b/prbt_hardware_support/test/unittests/unittest_operation_mode_setup_executor.cpp
@@ -299,7 +299,7 @@ INSTANTIATE_TEST_CASE_P(
     std::pair<OperationModes::_value_type, double>(OperationModes::UNKNOWN, 0.0),
     std::pair<OperationModes::_value_type, double>(OperationModes::T1,      0.1),
     std::pair<OperationModes::_value_type, double>(OperationModes::AUTO,    1.0)
-  )
+  ),
 );
 
 }  // namespace operation_mode_setup_executor_tests

diff --git a/prbt_support/test/urdf_tests.cpp b/prbt_support/test/urdf_tests.cpp
@@ -95,7 +95,7 @@ void URDFKinematicsTest::SetUp()
   testDataSet_.push_back(TestDataPoint("TestPos10", {      0,       0,       0,  0,       0,  M_PI_2}, Eigen::Vector3d(0,    0, L0+L1+L2+L3)));
 }
 
-INSTANTIATE_TEST_CASE_P(InstantiationName, URDFKinematicsTest, ::testing::Values(PARAM_MODEL));
+INSTANTIATE_TEST_CASE_P(InstantiationName, URDFKinematicsTest, ::testing::Values(PARAM_MODEL), );
 /**
  * \brief test the kinematics of urdf model
  *