In !7572, I remove #include <math.h> from vtkSetGet.h (which is included by all VTK class headers). The reason for the removal is modernization of the code, but of course it will cause some (easily fixed) compile errors in apps and libraries that use VTK.
Do people have any opinions on the best time to merge this change?
I think “small” inconvenience, but not quite “tiny”
For example it required the addition of #include <cmath> to around 20 .cxx files in Paraview, and will probably require a similar number of changes in Slicer.
I should add that one of my reasons for the change is that it hides math.h from the wrappers. The handling of system & compiler includes by the wrappers is imperfect, so moving them into the .cxx files where the wrappers don’t see them makes wrapping more robust.