Skip to content
This repository was archived by the owner on May 27, 2026. It is now read-only.

Commit c6bfa78

Browse files
hjmjohnsonclaude
andcommitted
COMP: Fix missing include path and nodiscard warning
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) <noreply@anthropic.com>
1 parent 09bed60 commit c6bfa78

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

include/itkContinuousBorderWarpImageFilter.hxx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,10 @@ ContinuousBorderWarpImageFilter<TInputImage, TOutputImage, TDisplacementField>::
7171
}
7272

7373
// project point into image region
74-
inputPtr->TransformPhysicalPointToContinuousIndex(point, contIndex);
74+
if (!inputPtr->TransformPhysicalPointToContinuousIndex(point, contIndex))
75+
{
76+
itkExceptionMacro("Physical point " << point << " is outside the image domain");
77+
}
7578

7679
for (unsigned int j = 0; j < ImageDimension; j++)
7780
{

src/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
include_directories("${VariationalRegistration_SOURCE_DIR}/include")
12
include_directories("${VariationalRegistration_SOURCE_DIR}/src")
23

34
# Test if ITK is built with FFTW. FFTWD is recommended and required for the

0 commit comments

Comments
 (0)