Skip to content

COMP: Coverity integer handling issues

Bill Lorensen requested to merge lorensen/vtk:Coverity_IntegerHandlingIssues into master

The SetClampMacros that use VTK_INT_MAX, VTK_UNSIGNED_LONG_MAX or VTK_ID_MAX as upper limits cause the Coverity warning: "operands don't affect result". The comparision of the upper linit is always false regardless of the values of its operands.

This patch sets the upper limits to VTK_INT_MAX - 1, VTK_UNSIGNED_LONG_MAX - 1 and VTK_ID_MAX - 1.

Merge request reports