COMP: Fix missing include path and nodiscard warning

hjmjohnson · claude · hjmjohnson · commit 8bedb8204f67 · 2026-04-10T18:11:09.000-05:00
Add the module's include/ directory to the src/ executables' include
path so that itkVariationalRegistrationMultiResolutionFilter.h and
other module headers are found during the CTest dashboard build.

Also fix the nodiscard warning on TransformPhysicalPointToContinuousIndex
in itkContinuousBorderWarpImageFilter.hxx by throwing an exception when
the physical point falls outside the image domain.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/include/itkContinuousBorderWarpImageFilter.hxx b/include/itkContinuousBorderWarpImageFilter.hxx
@@ -72,7 +72,10 @@ ContinuousBorderWarpImageFilter<TInputImage, TOutputImage, TDisplacementField>::
       }
 
       // project point into image region
-      inputPtr->TransformPhysicalPointToContinuousIndex(point, contIndex);
+      if (!inputPtr->TransformPhysicalPointToContinuousIndex(point, contIndex))
+      {
+        itkExceptionMacro("Physical point " << point << " is outside the image domain");
+      }
 
       for (unsigned int j = 0; j < ImageDimension; j++)
       {
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
@@ -1,3 +1,4 @@
+include_directories("${VariationalRegistration_SOURCE_DIR}/include")
 include_directories("${VariationalRegistration_SOURCE_DIR}/src")
 
 # Test if ITK is built with FFTW. FFTWD is recommended and required for the

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,10 @@ ContinuousBorderWarpImageFilter<TInputImage, TOutputImage, TDisplacementField>::`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`// project point into image region`
`75`		`- inputPtr->TransformPhysicalPointToContinuousIndex(point, contIndex);`
	`75`	`+ if (!inputPtr->TransformPhysicalPointToContinuousIndex(point, contIndex))`
	`76`	`+ {`
	`77`	`+ itkExceptionMacro("Physical point " << point << " is outside the image domain");`
	`78`	`+ }`
`76`	`79`
`77`	`80`	`for (unsigned int j = 0; j < ImageDimension; j++)`
`78`	`81`	`{`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+include_directories("${VariationalRegistration_SOURCE_DIR}/include")`
`1`	`2`	`include_directories("${VariationalRegistration_SOURCE_DIR}/src")`
`2`	`3`
`3`	`4`	`# Test if ITK is built with FFTW. FFTWD is recommended and required for the`