Use constexpr in ImportImageFilter

blowekamp · blowekamp · commit aeca49fbf112 · 2024-10-08T13:47:54.000-04:00
diff --git a/Code/IO/include/sitkImportImageFilter.h b/Code/IO/include/sitkImportImageFilter.h
@@ -118,11 +118,7 @@ class SITKIO_EXPORT ImportImageFilter : public ImageReaderBase
   // components per pixel needs to be set, otherwise the method
   // does not exist. This is done with the enable if idiom.
   template <class TImageType>
-  typename std::enable_if<!IsVector<TImageType>::Value>::type
-  SetNumberOfComponentsOnImage(TImageType *)
-  {}
-  template <class TImageType>
-  typename std::enable_if<IsVector<TImageType>::Value>::type
+  void
   SetNumberOfComponentsOnImage(TImageType *);
 
 private:
diff --git a/Code/IO/src/sitkImportImageFilter.cxx b/Code/IO/src/sitkImportImageFilter.cxx
@@ -536,22 +536,15 @@ ImportImageFilter::ExecuteInternal()
                                                TheContainerWillTakeCareOfDeletingTheMemoryBuffer);
 
 
-  //
-  // Meta-programmed method to set the number of components if a
-  // vector image
-  //
-  this->SetNumberOfComponentsOnImage(image.GetPointer());
+  // set the number of components if a vector image
+  if constexpr (IsVector<ImageType>::Value)
+  {
+    image->SetNumberOfComponentsPerPixel(m_NumberOfComponentsPerPixel);
+  }
 
   // This line must be the last line in the function to prevent a deep
   // copy caused by a implicit sitk::MakeUnique
   return Image(image);
 }
 
-template <class TFilterType>
-typename std::enable_if<IsVector<TFilterType>::Value>::type
-ImportImageFilter::SetNumberOfComponentsOnImage(TFilterType * image)
-{
-  image->SetNumberOfComponentsPerPixel(m_NumberOfComponentsPerPixel);
-}
-
 } // namespace itk::simple
