add operator!= for duration (#1236) (#1277)

ivanpauno · Jannik Abbenseth · web-flow · commit d27ca8ee8598 · 2020-08-18T10:28:23.000-03:00
Signed-off-by: Jannik Abbenseth &lt;jannik.abbenseth@ipa.fraunhofer.de&gt;
Signed-off-by: Ivan Santiago Paunovic &lt;ivanpauno@ekumenlabs.com&gt;

Co-authored-by: Jannik Abbenseth &lt;ipa-jba@users.noreply.github.com&gt;
diff --git a/rclcpp/include/rclcpp/duration.hpp b/rclcpp/include/rclcpp/duration.hpp
@@ -60,6 +60,9 @@ class RCLCPP_PUBLIC Duration
   bool
   operator==(const rclcpp::Duration & rhs) const;
 
+  bool
+  operator!=(const rclcpp::Duration & rhs) const;
+
   bool
   operator<(const rclcpp::Duration & rhs) const;
 
diff --git a/rclcpp/src/rclcpp/duration.cpp b/rclcpp/src/rclcpp/duration.cpp
@@ -95,6 +95,12 @@ Duration::operator==(const rclcpp::Duration & rhs) const
   return rcl_duration_.nanoseconds == rhs.rcl_duration_.nanoseconds;
 }
 
+bool
+Duration::operator!=(const rclcpp::Duration & rhs) const
+{
+  return rcl_duration_.nanoseconds != rhs.rcl_duration_.nanoseconds;
+}
+
 bool
 Duration::operator<(const rclcpp::Duration & rhs) const
 {
diff --git a/rclcpp/test/test_duration.cpp b/rclcpp/test/test_duration.cpp
@@ -45,6 +45,7 @@ TEST(TestDuration, operators) {
   EXPECT_TRUE(old <= young);
   EXPECT_TRUE(young >= old);
   EXPECT_FALSE(young == old);
+  EXPECT_TRUE(young != old);
 
   rclcpp::Duration add = old + young;
   EXPECT_EQ(add.nanoseconds(), (rcl_duration_value_t)(old.nanoseconds() + young.nanoseconds()));

Original file line number	Diff line number	Diff line change
`@@ -95,6 +95,12 @@ Duration::operator==(const rclcpp::Duration & rhs) const`
`95`	`95`	`return rcl_duration_.nanoseconds == rhs.rcl_duration_.nanoseconds;`
`96`	`96`	`}`
`97`	`97`
	`98`	`+bool`
	`99`	`+Duration::operator!=(const rclcpp::Duration & rhs) const`
	`100`	`+{`
	`101`	`+ return rcl_duration_.nanoseconds != rhs.rcl_duration_.nanoseconds;`
	`102`	`+}`
	`103`	`+`
`98`	`104`	`bool`
`99`	`105`	`Duration::operator<(const rclcpp::Duration & rhs) const`
`100`	`106`	`{`