Fix warnings on Jammy (#464)

* Add .cc extension to suppress CMake warnings (#428) Signed-off-by: Michael Carroll <michael@openrobotics.org> * Fix -Wfree-nonheap-object Signed-off-by: Louise Poubel <louise@openrobotics.org> Signed-off-by: Michael Carroll <michael@openrobotics.org> Signed-off-by: Louise Poubel <louise@openrobotics.org> Co-authored-by: Michael Carroll <michael@openrobotics.org>
gazebosim · Aug 11, 2022 · eef3853 · eef3853
1 parent c9b9fc0
commit eef3853
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 13 deletions.
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
@@ -16,17 +16,17 @@ set (sources
 )
 
 set (gtest_sources
-  Application_TEST
-  Conversions_TEST
-  Dialog_TEST
-  DragDropModel_TEST
-  Helpers_TEST
-  GuiEvents_TEST
-  ign_TEST
-  MainWindow_TEST
-  PlottingInterface_TEST
-  Plugin_TEST
-  SearchModel_TEST
+  Application_TEST.cc
+  Conversions_TEST.cc
+  Dialog_TEST.cc
+  DragDropModel_TEST.cc
+  Helpers_TEST.cc
+  GuiEvents_TEST.cc
+  ign_TEST.cc
+  MainWindow_TEST.cc
+  PlottingInterface_TEST.cc
+  Plugin_TEST.cc
+  SearchModel_TEST.cc
 )
 
 if (MSVC)

diff --git a/src/PlottingInterface_TEST.cc b/src/PlottingInterface_TEST.cc
@@ -172,8 +172,7 @@ TEST(PlottingInterfaceTest, IGN_UTILS_TEST_DISABLED_ON_WIN32(Transport))
 
   auto transport = Transport();
 
-  double time = 10;
-  std::shared_ptr<double> timeRef(&time);
+  auto timeRef = std::make_shared<double>(10);
 
   transport.Subscribe("/collision_topic", "pose-position-x", 1, timeRef);
   transport.Subscribe("/collision_topic", "pose-position-z", 1, timeRef);