STYLE: Let Point, RGBPixel, RGBAPixel operator== just call the BaseArray

Similar to the implementation of `Vector::operator==`.
InsightSoftwareConsortium · Mar 1, 2024 · 28f84ab · 28f84ab
1 parent 613c8f4
commit 28f84ab
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 23 deletions.
diff --git a/Modules/Core/Common/include/itkPoint.h b/Modules/Core/Common/include/itkPoint.h
@@ -137,13 +137,7 @@ class ITK_TEMPLATE_EXPORT Point : public FixedArray<TCoordRep, VPointDimension>
   bool
   operator==(const Self & pt) const
   {
-    bool same = true;
-
-    for (unsigned int i = 0; i < VPointDimension && same; ++i)
-    {
-      same = (Math::ExactlyEquals((*this)[i], pt[i]));
-    }
-    return same;
+    return this->BaseArray::operator==(pt);
   }
 
   ITK_UNEQUAL_OPERATOR_MEMBER_FUNCTION(Self);

diff --git a/Modules/Core/Common/include/itkRGBAPixel.hxx b/Modules/Core/Common/include/itkRGBAPixel.hxx
@@ -131,14 +131,7 @@ template <typename T>
 bool
 RGBAPixel<T>::operator==(const Self & r) const
 {
-  for (unsigned int i = 0; i < 4; ++i)
-  {
-    if ((*this)[i] != r[i])
-    {
-      return false;
-    }
-  }
-  return true;
+  return this->BaseArray::operator==(r);
 }
 
 template <typename T>

diff --git a/Modules/Core/Common/include/itkRGBPixel.hxx b/Modules/Core/Common/include/itkRGBPixel.hxx
@@ -131,14 +131,7 @@ template <typename T>
 bool
 RGBPixel<T>::operator==(const Self & r) const
 {
-  for (unsigned int i = 0; i < 3; ++i)
-  {
-    if (Math::NotExactlyEquals((*this)[i], r[i]))
-    {
-      return false;
-    }
-  }
-  return true;
+  return this->BaseArray::operator==(r);
 }
 
 template <typename T>