Merge pull request InsightSoftwareConsortium#5794 from N-Dekker/ImageConstIterator-GetIndex-remove-const-and-static_cast

hjmjohnson · web-flow · commit 00844a245d77 · 2026-02-11T20:51:51.000-06:00
Remove const from return type of `ImageConstIterator::GetIndex()`, remove static_cast
diff --git a/Modules/Core/Common/include/itkImageConstIterator.h b/Modules/Core/Common/include/itkImageConstIterator.h
@@ -299,10 +299,10 @@ class ITK_TEMPLATE_EXPORT ImageConstIterator
    * This causes the index to be calculated from pointer arithmetic and is
    * therefore an expensive operation.
    * \sa SetIndex */
-  [[nodiscard]] const IndexType
+  [[nodiscard]] IndexType
   GetIndex() const
   {
-    return m_Image->ComputeIndex(static_cast<OffsetValueType>(m_Offset));
+    return m_Image->ComputeIndex(m_Offset);
   }
 
   /** Set the index. No bounds checking is performed.

Original file line number	Diff line number	Diff line change
`@@ -299,10 +299,10 @@ class ITK_TEMPLATE_EXPORT ImageConstIterator`
`299`	`299`	`* This causes the index to be calculated from pointer arithmetic and is`
`300`	`300`	`* therefore an expensive operation.`
`301`	`301`	`* \sa SetIndex */`
`302`		`- [[nodiscard]] const IndexType`
	`302`	`+ [[nodiscard]] IndexType`
`303`	`303`	`GetIndex() const`
`304`	`304`	`{`
`305`		`- return m_Image->ComputeIndex(static_cast<OffsetValueType>(m_Offset));`
	`305`	`+ return m_Image->ComputeIndex(m_Offset);`
`306`	`306`	`}`
`307`	`307`
`308`	`308`	`/** Set the index. No bounds checking is performed.`