search

borglab / gtsam

GTSAM is a library of C++ classes that implement smoothing and mapping (SAM) in robotics and vision, using factor graphs and Bayes networks as the underlying computing paradigm rather than sparse matrices.
http://gtsam.org
Other
2.6k stars 764 forks source link

Warning in SmartRangeExample_plaza1 #38

Closed dellaert closed 5 years ago

dellaert commented 5 years ago

Recent travis run revealed: 

[ 96%] Building CXX object gtsam_unstable/examples/CMakeFiles/SmartRangeExample_plaza1.dir/SmartRangeExample_plaza1.cpp.o
In file included from /home/travis/build/borglab/gtsam/gtsam/3rdparty/Eigen/Eigen/Core:450:0,
                 from /home/travis/build/borglab/gtsam/gtsam/3rdparty/Eigen/Eigen/Dense:1,
                 from /home/travis/build/borglab/gtsam/gtsam/base/OptionalJacobian.h:22,
                 from /home/travis/build/borglab/gtsam/gtsam/base/Matrix.h:25,
                 from /home/travis/build/borglab/gtsam/gtsam/base/Manifold.h:22,
                 from /home/travis/build/borglab/gtsam/gtsam/geometry/BearingRange.h:21,
                 from /home/travis/build/borglab/gtsam/gtsam/geometry/Pose2.h:23,
                 from /home/travis/build/borglab/gtsam/gtsam_unstable/examples/SmartRangeExample_plaza1.cpp:20:
/home/travis/build/borglab/gtsam/gtsam/3rdparty/Eigen/Eigen/src/Core/DenseStorage.h: In member function ‘gtsam::Point2 gtsam::SmartRangeFactor::triangulate(const gtsam::Values&) const’:
/home/travis/build/borglab/gtsam/gtsam/3rdparty/Eigen/Eigen/src/Core/DenseStorage.h:194:66: warning: ‘*((void*)& bestCircle2 +16)’ may be used uninitialized in this function [-Wmaybe-uninitialized]
     DenseStorage(const DenseStorage& other) : m_data(other.m_data) {
                                                                  ^
In file included from /home/travis/build/borglab/gtsam/gtsam_unstable/examples/SmartRangeExample_plaza1.cpp:43:0:
/home/travis/build/borglab/gtsam/gtsam_unstable/slam/SmartRangeFactor.h:103:30: note: ‘*((void*)& bestCircle2 +16)’ was declared here
     boost::optional<Circle2> bestCircle2;
                              ^
dellaert commented 5 years ago

Fixed in PR #64