PERF: Use string macro for exceptions

Add itkExceptionStringMacro which remove the usage of ostringstream,
and passes argument directly to ExceptionObject.

Author: blowekamp

Modules/Bridge/VTK/include/itkVTKImageExport.hxx

@@ -97,7 +97,7 @@ VTKImageExport<TInputImage>::VTKImageExport()
   }
   else
   {
-    itkExceptionMacro("Type currently not supported");
+    itkExceptionStringMacro("Type currently not supported");
   }
 }
 
@@ -141,7 +141,7 @@ VTKImageExport<TInputImage>::WholeExtentCallback()
 
   if (!input)
   {
-    itkExceptionMacro("Need to set an input");
+    itkExceptionStringMacro("Need to set an input");
   }
 
   const InputRegionType region = input->GetLargestPossibleRegion();
@@ -176,7 +176,7 @@ VTKImageExport<TInputImage>::SpacingCallback()
 
   if (!input)
   {
-    itkExceptionMacro("Need to set an input");
+    itkExceptionStringMacro("Need to set an input");
   }
 
   const typename TInputImage::SpacingType & spacing = input->GetSpacing();
@@ -233,7 +233,7 @@ VTKImageExport<TInputImage>::OriginCallback()
 
   if (!input)
   {
-    itkExceptionMacro("Need to set an input");
+    itkExceptionStringMacro("Need to set an input");
   }
 
   const typename TInputImage::PointType & origin = input->GetOrigin();
@@ -290,7 +290,7 @@ VTKImageExport<TInputImage>::DirectionCallback()
 
   if (!input)
   {
-    itkExceptionMacro("Need to set an input");
+    itkExceptionStringMacro("Need to set an input");
   }
 
   const typename TInputImage::DirectionType & direction = input->GetDirection();
@@ -363,7 +363,7 @@ VTKImageExport<TInputImage>::PropagateUpdateExtentCallback(int * extent)
   const InputImagePointer input = this->GetInput();
   if (!input)
   {
-    itkExceptionMacro("Need to set an input");
+    itkExceptionStringMacro("Need to set an input");
   }
 
   input->SetRequestedRegion(region);
@@ -383,7 +383,7 @@ VTKImageExport<TInputImage>::DataExtentCallback()
 
   if (!input)
   {
-    itkExceptionMacro("Need to set an input");
+    itkExceptionStringMacro("Need to set an input");
   }
 
   const InputRegionType region = input->GetBufferedRegion();
@@ -416,7 +416,7 @@ VTKImageExport<TInputImage>::BufferPointerCallback()
 
   if (!input)
   {
-    itkExceptionMacro("Need to set an input");
+    itkExceptionStringMacro("Need to set an input");
   }
 
   return input->GetBufferPointer();

Modules/Bridge/VTK/include/itkVTKImageImport.hxx

@@ -92,7 +92,7 @@ VTKImageImport<TOutputImage>::VTKImageImport()
   }
   else
   {
-    itkExceptionMacro("Type currently not supported");
+    itkExceptionStringMacro("Type currently not supported");
   }
 }
 
@@ -107,7 +107,7 @@ VTKImageImport<TOutputImage>::PropagateRequestedRegion(DataObject * outputPtr)
 
   if (!output)
   {
-    itkExceptionMacro("Downcast from DataObject to my Image type failed.");
+    itkExceptionStringMacro("Downcast from DataObject to my Image type failed.");
   }
   Superclass::PropagateRequestedRegion(output);
   if (m_PropagateUpdateExtentCallback)

Modules/Bridge/VTK/src/itkVTKImageExportBase.cxx

@@ -131,7 +131,7 @@ VTKImageExportBase::PipelineModifiedCallback()
 
   if (!input)
   {
-    itkExceptionMacro("Need to set an input");
+    itkExceptionStringMacro("Need to set an input");
   }
 
   ModifiedTimeType pipelineMTime = input->GetPipelineMTime();
@@ -169,7 +169,7 @@ VTKImageExportBase::UpdateDataCallback()
 
   if (!input)
   {
-    itkExceptionMacro("Need to set an input");
+    itkExceptionStringMacro("Need to set an input");
   }
 
   // Notify start event observers

Modules/Core/Common/include/itkExtractImageFilter.h

@@ -202,7 +202,7 @@ class ITK_TEMPLATE_EXPORT ExtractImageFilter : public InPlaceImageFilter<TInputI
         break;
       case DirectionCollapseStrategyEnum::DIRECTIONCOLLAPSETOUNKOWN:
       default:
-        itkExceptionMacro("Invalid Strategy Chosen for itk::ExtractImageFilter");
+        itkExceptionStringMacro("Invalid Strategy Chosen for itk::ExtractImageFilter");
     }
 
     this->m_DirectionCollapseStrategy = choosenStrategy;

Modules/Core/Common/include/itkExtractImageFilter.hxx

@@ -196,7 +196,7 @@ ExtractImageFilter<TInputImage, TOutputImage>::GenerateOutputInformation()
       {
         if (vnl_determinant(outputDirection.GetVnlMatrix()) == 0.0)
         {
-          itkExceptionMacro("Invalid submatrix extracted for collapsed direction.");
+          itkExceptionStringMacro("Invalid submatrix extracted for collapsed direction.");
         }
       }
       break;

Modules/Core/Common/include/itkFrustumSpatialFunction.hxx

@@ -106,7 +106,7 @@ FrustumSpatialFunction<VDimension, TInput>::Evaluate(const InputType & position)
   }
   else
   {
-    itkExceptionMacro("Rotation plane not set or set to an unsupported value!");
+    itkExceptionStringMacro("Rotation plane not set or set to an unsupported value!");
   }
 }
 

Modules/Core/Common/include/itkGaussianOperator.h

@@ -90,7 +90,7 @@ class ITK_TEMPLATE_EXPORT GaussianOperator : public NeighborhoodOperator<TPixel,
   {
     if (max_error >= 1 || max_error <= 0)
     {
-      itkExceptionMacro("Maximum Error Must be in the range [ 0.0 , 1.0 ]");
+      itkExceptionStringMacro("Maximum Error Must be in the range [ 0.0 , 1.0 ]");
     }
 
     m_MaximumError = max_error;

Modules/Core/Common/include/itkImageDuplicator.hxx

@@ -29,7 +29,7 @@ ImageDuplicator<TInputImage>::Update()
 {
   if (!m_InputImage)
   {
-    itkExceptionMacro("Input image has not been connected");
+    itkExceptionStringMacro("Input image has not been connected");
   }
 
   // Update only if the input image has been modified

Modules/Core/Common/include/itkImageSource.hxx

@@ -107,7 +107,7 @@ ImageSource<TOutputImage>::GraftOutput(const DataObjectIdentifierType & key, Dat
 {
   if (!graft)
   {
-    itkExceptionMacro("Requested to graft output that is a nullptr pointer");
+    itkExceptionStringMacro("Requested to graft output that is a nullptr pointer");
   }
 
   // we use the process object method since all out output may not be

Modules/Core/Common/include/itkPointSetBase.hxx

@@ -66,7 +66,7 @@ PointSetBase<TPointsContainer>::SetPoints(PointsVectorContainer * points)
   itkDebugMacro("setting Points container to " << points);
   if (points->Size() % PointDimension != 0)
   {
-    itkExceptionMacro("Number of entries in given 1d array incompatible with the point dimension");
+    itkExceptionStringMacro("Number of entries in given 1d array incompatible with the point dimension");
   }
 
   // Note: this cast is unsafe. It may lead to undefined behavior.
@@ -87,7 +87,7 @@ PointSetBase<TPointsContainer>::SetPointsByCoordinates(const std::vector<Coordin
 
   if (numberOfCoordinates % PointDimension != 0)
   {
-    itkExceptionMacro("Number of specified coordinates incompatible with the point dimension");
+    itkExceptionStringMacro("Number of specified coordinates incompatible with the point dimension");
   }
 
   const size_t numberOfPoints = numberOfCoordinates / PointDimension;
@@ -192,7 +192,7 @@ PointSetBase<TPointsContainer>::GetPoint(PointIdentifier ptId) const -> PointTyp
    */
   if (!m_PointsContainer)
   {
-    itkExceptionMacro("Point container doesn't exist.");
+    itkExceptionStringMacro("Point container doesn't exist.");
   }
 
   /**