Open jhlegarreta opened 2 days ago
Realized about these when resuming work on #3908. By no means this PR is intended to be extensive in the changes: separate, incremental PRs as code is inspected can be done.
/azp ITK.macOS
ITKmacOS
failures are unrelated:
ld: warning: ignoring duplicate libraries: '-lm'
Tests are passing: https://dev.azure.com/itkrobotmacos/ITK.macOS/_build/results?buildId=11648&view=ms.vss-test-web.build-test-results-tab
https://open.cdash.org/viewBuildError.php?type=1&buildid=9723501
Same applies to ITK.macOS.Python
:
https://dev.azure.com/itkrobotmacospython/ITK.macOS.Python/_build/results?buildId=10403&view=ms.vss-test-web.build-test-results-tab
Improve ivar printing in
PrintSelf
methods:itkPrintSelfObjectMacro
macro to avoid boilerplate code when printing smart pointers/objects that can be null pointers.({ivar} ? "On" : "Off")
recipe in agreement with the ITK SW Guide.std::string
types ivars are empty to conditionally print their contents; rely on the ostream insertion operator overload to print the appropriate content.std::endl
instead of\n
to add a new line for the sake of consistency.Take advantage of the commit to print ivars that had been missed in the modified classes.
PR Checklist