Fix a couple of warnings when compiling on Ubuntu 24.04. (#4)

Signed-off-by: Chris Lalancette <[email protected]>
clalancette · Jun 6, 2024 · 2a9215d · 2a9215d
1 parent 57ebc5e
commit 2a9215d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -34,7 +34,7 @@ if(SOPHUS_MASTER_PROJECT)
   ELSEIF("${CMAKE_CXX_COMPILER_ID}" STREQUAL "GNU")
     SET(CMAKE_CXX_FLAGS_DEBUG  "-O0 -g")
     SET(CMAKE_CXX_FLAGS_RELEASE "-O3")
-    SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -std=c++14 -Wno-deprecated-declarations -ftemplate-backtrace-limit=0")
+    SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -std=c++14 -Wno-deprecated-declarations -ftemplate-backtrace-limit=0 -Wno-array-bounds")
     SET(CMAKE_CXX_FLAGS_COVERAGE "${CMAKE_CXX_FLAGS_DEBUG} --coverage -fno-inline -fno-inline-small-functions -fno-default-inline")
     SET(CMAKE_EXE_LINKER_FLAGS_COVERAGE "${CMAKE_EXE_LINKER_FLAGS_DEBUG} --coverage")
     SET(CMAKE_SHARED_LINKER_FLAGS_COVERAGE "${CMAKE_SHARED_LINKER_FLAGS_DEBUG} --coverage")

diff --git a/test/ceres/tests.hpp b/test/ceres/tests.hpp
@@ -243,9 +243,9 @@ struct LieGroupCeresTests {
     for (size_t i = 0; i < num_vertices; ++i)
       for (size_t j = i + 1; j < num_vertices; ++j) {
         LieGroupd diff = V[i].inverse() * V[j];
-        const auto delta_log =
-            Random<typename LieGroupd::Tangent>::sample(rng) *
-            sigma_observation_elementwise;
+        typename LieGroupd::Tangent sample =
+            Random<typename LieGroupd::Tangent>::sample(rng);
+        const auto delta_log = sample * sigma_observation_elementwise;
         const auto delta = LieGroupd::exp(delta_log);
         ceres::CostFunction* cost =
             new ceres::AutoDiffCostFunction<TestGraphFunctor, LieGroupd::DoF,