Fix GCC 13 issues

borglab · Jan 21, 2025 · bc19073 · bc19073
1 parent c3908c8
commit bc19073
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 4 deletions.
diff --git a/examples/TriangulationLOSTExample.cpp b/examples/TriangulationLOSTExample.cpp
@@ -123,9 +123,9 @@ int main(int argc, char* argv[]) {
 
   double rank_tol = 1e-9;
   std::shared_ptr<Cal3_S2> calib = std::make_shared<Cal3_S2>();
-  std::chrono::nanoseconds durationDLT;
-  std::chrono::nanoseconds durationDLTOpt;
-  std::chrono::nanoseconds durationLOST;
+  std::chrono::nanoseconds durationDLT{};
+  std::chrono::nanoseconds durationDLTOpt{};
+  std::chrono::nanoseconds durationLOST{};
 
   for (int i = 0; i < nrTrials; i++) {
     Point2Vector noisyMeasurements =

diff --git a/gtsam/geometry/BearingRange.h b/gtsam/geometry/BearingRange.h
@@ -26,6 +26,11 @@
 #endif
 #include <iostream>
 
+#ifdef __GNUC__ && !defined(__clang__)
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Warray-bounds"
+#endif
+
 namespace gtsam {
 
 // Forward declaration of Bearing functor which should be of A1*A2 -> return_type
@@ -162,7 +167,7 @@ struct BearingRange {
   /// @}
 
   // Alignment, see https://eigen.tuxfamily.org/dox/group__TopicStructHavingEigenMembers.html
-  inline constexpr static auto NeedsToAlign = (sizeof(B) % 16) == 0 || (sizeof(R) % 16) == 0;
+  inline constexpr static auto NeedsToAlign = true;
 public:
   GTSAM_MAKE_ALIGNED_OPERATOR_NEW_IF(NeedsToAlign)
 };
@@ -203,3 +208,7 @@ struct HasRange {
 };
 
 }  // namespace gtsam
+
+#ifdef __GNUC__
+#pragma GCC diagnostic pop
+#endif
diff --git a/gtsam/geometry/FundamentalMatrix.cpp b/gtsam/geometry/FundamentalMatrix.cpp
@@ -9,6 +9,10 @@
 #include <gtsam/geometry/FundamentalMatrix.h>
 #include <gtsam/geometry/Point2.h>
 
+#ifdef __GNUC__
+#pragma GCC diagnostic ignored "-Wmaybe-uninitialized"
+#endif
+
 namespace gtsam {
 
 //*************************************************************************