12345qiupeng
/
gtsam
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix equality operator for Point2, and add test that would have caught this bug

release/4.3a0
cbeall3 2015-05-26 12:25:33 -04:00
parent 913affc755
commit 3d99c24d82
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
gtsam/geometry/Point2.h
View File

@ -148,7 +148,7 @@ public:
/// @{ /// @{
/// equality /// equality
inline bool operator ==(const Point2& q) const {return x_==q.x_ && q.y_==q.y_;} inline bool operator ==(const Point2& q) const {return x_==q.x_ && y_==q.y_;}
/// get x /// get x
double x() const {return x_;} double x() const {return x_;}

6
gtsam/geometry/tests/testPoint2.cpp
View File

@ -61,6 +61,12 @@ TEST(Point2, constructor) {
EXPECT(assert_equal(p1, p2)); EXPECT(assert_equal(p1, p2));
} }
/* ************************************************************************* */
TEST(Point2, equality) {
Point2 p1(1, 2), p2(1,3);
EXPECT(!(p1 == p2));
}
/* ************************************************************************* */ /* ************************************************************************* */
TEST(Point2, Lie) { TEST(Point2, Lie) {
Point2 p1(1, 2), p2(4, 5); Point2 p1(1, 2), p2(4, 5);