Skip to content

Commit

Permalink
COMP: Fix compile warnings: discarding return value with [[nodiscard]]
Browse files Browse the repository at this point in the history
Example message:

C:\Dev\ANTs\ImageRegistration\itkAvantsMutualInformationRegistrationFunction.cxx(368,54): warning C4834: discarding return value of function with [[nodiscard]] attribute [C:\Dev\ANTs-SB\ANTS-build\Examples\l_ANTS.vcxproj]
  • Loading branch information
dzenanz committed Jan 11, 2024
1 parent 2f7be46 commit 82453ae
Show file tree
Hide file tree
Showing 7 changed files with 9 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -283,7 +283,7 @@ LandmarkBasedDisplacementFieldTransformInitializer(int argc, char * argv[])
}

itk::ContinuousIndex<double, ImageDimension> fixedCidx;
fixedImage->TransformPhysicalPointToContinuousIndex(fixedPhysicalPoint, fixedCidx);
fixedCidx = fixedImage->TransformPhysicalPointToContinuousIndex(fixedPhysicalPoint);

typename DisplacementFieldType::PointType fieldPoint;
parametricInputImage->TransformContinuousIndexToPhysicalPoint(fixedCidx, fieldPoint);
Expand Down
4 changes: 2 additions & 2 deletions Examples/antsLandmarkBasedTransformInitializer.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -372,10 +372,10 @@ InitializeBSplineTransform(int argc, char * argv[])
}

itk::ContinuousIndex<double, ImageDimension> fixedCidx;
fixedImage->TransformPhysicalPointToContinuousIndex(fixedPhysicalPoint, fixedCidx);
fixedCidx = fixedImage->TransformPhysicalPointToContinuousIndex(fixedPhysicalPoint);

typename DisplacementFieldType::PointType fieldPoint;
parametricInputImage->TransformContinuousIndexToPhysicalPoint(fixedCidx, fieldPoint);
fieldPoint = parametricInputImage->TransformContinuousIndexToPhysicalPoint(fixedCidx);

fieldPoints->SetPoint(count, fieldPoint);
fieldPoints->SetPointData(count, vector);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -365,7 +365,7 @@ AvantsMutualInformationRegistrationFunction<TFixedImage, TMovingImage, TDisplace
this->ComputeJointPDFPoint(fixedImageValue, movingImageValue, jointPDFpoint);
JointPDFIndexType jointPDFIndex;
jointPDFIndex.Fill(0);
this->m_JointPDF->TransformPhysicalPointToIndex(jointPDFpoint, jointPDFIndex);
jointPDFIndex = this->m_JointPDF->TransformPhysicalPointToIndex(jointPDFpoint);
this->m_JointPDF->SetPixel(jointPDFIndex, this->m_JointPDF->GetPixel(jointPDFIndex) + 1);

++nSamples;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ HistogramParzenWindowsListSampleFunction<TListSample, TOutput, TCoordRep>::SetIn
point[0] = inputMeasurement[d];

ContinuousIndex<double, 1> cidx;
this->m_HistogramImages[d]->TransformPhysicalPointToContinuousIndex(point, cidx);
cidx = this->m_HistogramImages[d]->TransformPhysicalPointToContinuousIndex(point);

typename HistogramImageType::IndexType idx;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ JointHistogramParzenShapeAndOrientationListSampleFunction<TListSample, TOutput,
shapePoint[1] = eigenvalue2 * (this->m_NumberOfShapeJointHistogramBins - 1);

ContinuousIndex<double, 2> shapeCidx;
this->m_JointHistogramImages[0]->TransformPhysicalPointToContinuousIndex(shapePoint, shapeCidx);
shapeCidx = this->m_JointHistogramImages[0]->TransformPhysicalPointToContinuousIndex(shapePoint);

typedef typename JointHistogramImageType::IndexType JointHistogramImageIndexType;
JointHistogramImageIndexType shapeIdx;
Expand Down Expand Up @@ -272,7 +272,7 @@ JointHistogramParzenShapeAndOrientationListSampleFunction<TListSample, TOutput,
static_cast<RealType>(this->m_NumberOfOrientationJointHistogramBins - 1));

ContinuousIndex<double, 2> orientCidx;
this->m_JointHistogramImages[whichHistogram]->TransformPhysicalPointToContinuousIndex(orientPoint, orientCidx);
orientCidx = this->m_JointHistogramImages[whichHistogram]->TransformPhysicalPointToContinuousIndex(orientPoint);

typedef typename JointHistogramImageType::IndexType JointHistogramImageIndexType;
JointHistogramImageIndexType orientIdx;
Expand Down
2 changes: 1 addition & 1 deletion Utilities/itkSurfaceImageCurvature.hxx
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ SurfaceImageCurvature<TSurface>::FindEuclideanNeighborhood(
tempp[0] = rootpoint[0];
tempp[1] = rootpoint[1];
tempp[2] = rootpoint[2];
this->m_FunctionImage->TransformPhysicalPointToIndex(tempp, oindex);
oindex = this->m_FunctionImage->TransformPhysicalPointToIndex(tempp);
for (unsigned int i = 0; i < ImageDimension; i++)
{
rad[i] = (long)(m_NeighborhoodRadius/this->m_Spacing[i]+0.5);
Expand Down
2 changes: 1 addition & 1 deletion Utilities/itkWarpImageMultiTransformFilter.hxx
Original file line number Diff line number Diff line change
Expand Up @@ -540,7 +540,7 @@ WarpImageMultiTransformFilter<TInputImage, TOutputImage, TDisplacementField, TTr
// use continous coordinates
typename DefaultVectorInterpolatorType::ContinuousIndexType contind;
// use ITK implementation to use orientation header
fieldPtr->TransformPhysicalPointToContinuousIndex(point1, contind);
contind = fieldPtr->TransformPhysicalPointToContinuousIndex(point1);

isinside = fieldPtr->GetLargestPossibleRegion().IsInside(contind);

Expand Down

0 comments on commit 82453ae

Please sign in to comment.