Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Nonunit/Review/test compile errors introduced by pull request 4000 #4004

Conversation

N-Dekker
Copy link
Contributor

@N-Dekker N-Dekker commented Apr 7, 2023

Fixed compile errors saying:

error C2065: 'cindex': undeclared identifier

and:

error C2039: 'ContinuousIndexType': is not a member of 'itk::ContinuousIndex'

These compile errors were introduced by pull request #4000 commit 7cda5ba "COMP: Fix TransformPhysicalPointToContinuousIndex nodiscard warnings"

Fixed compile errors saying:

> error C2039: 'ContinuousIndexType': is not a member of 'itk::ContinuousIndex'

These compile errors were introduced by pull request InsightSoftwareConsortium#4000
commit 7cda5ba
"COMP: Fix TransformPhysicalPointToContinuousIndex `nodiscard` warnings"
Fixed compile errors saying:

> error C2065: 'cindex': undeclared identifier

These compile errors were introduced by pull request InsightSoftwareConsortium#4000
commit 7cda5ba
"COMP: Fix TransformPhysicalPointToContinuousIndex `nodiscard` warnings"
@github-actions github-actions bot added the type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct label Apr 7, 2023
@N-Dekker N-Dekker changed the title Fix Nonunit/Review/test compile error introduced by pull request 4000 Fix Nonunit/Review/test compile errors introduced by pull request 4000 Apr 7, 2023
@N-Dekker N-Dekker marked this pull request as ready for review April 7, 2023 13:11
@hjmjohnson hjmjohnson merged commit 1a1281c into InsightSoftwareConsortium:master Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants