Added VTK_ID_MIN and VTK_ID_MAX constants

Added VTK_ID_MIN constant, which did not exist before.
(Nothing uses it yet).

Added VTK_ID_MAX constant, which replaces VTK_LARGE_ID, which is
now only #defined if VTK_LEGACY_REMOVE is not.

These MIN/MAX names are more consistent with
existing constants like VTK_INT_MIN / VTK_INT_MAX.

Change-Id: Idafed9ce29df2e43d3318a6ad0c5d5246944f062
Replaced "-VTK_LARGE_ID-1" with "VTK_ID_MIN"

Replaced all usages of "-VTK_LARGE_ID-1" with
the new "VTK_ID_MIN" constant.

More readable.

Change-Id: I759214821076ebbf53cd749a78dd7baf7e717ac0
Replaced all "VTK_LARGE_ID" with "VTK_ID_MAX"

The constant "VTK_LARGE_ID" was deprecated
and replaced with "VTK_ID_MAX" (for consistency
in naming).

Change-Id: Id3b9d0155b451f95e2d6242fe463fde793126aa0
Fixed 64-to-32 truncation warning

Implicit cast was truncating long to int; just use long.
Also change from float to double to reduce chance of overflow.

Change-Id: I8f15e29f52e14ab746996f319e35ff1cadda2102
Removed superfluous semi-colon

Fixes -Wextra-semi warning that was repeated often since
this is a public header.

Change-Id: I5f21d0e15a1fcac655a019057ced8f03399c17ef
Fixed Wcast-align warning by removing cast

Removed unnecessary cast of malloc's result
(void*) to char*.

Change-Id: I715bfcd16ca314ec2d47a2d7a7e43bd091b3a8de
Removed unused macro DICOM_DBG_MSG

Found this dead code using -Wunused-macros. It appears nowhere else in the codebase.

Change-Id: I8d4bd03d0d0ac4abd0bf20201a60ffd4a3b30da2
Removed unused vtkTrilinFuncMacro macro

Macro unused in this file; already exists in vtkVolumeRaycastMapper.h.
Probably refactoring leftovers.

Change-Id: I4742a73451e20924240f97b73bd3d9341c99210b
Removed unused MAX_LIGHTS macro

Macro unused in these files; already exists in vtkXOpenGLRenderWindow.h.
Probably refactoring leftovers.

Change-Id: I9d43688833469ddb753db754dd1b65c0fc4686a0
Removed unused VTK_RINT macro

Macro unused in this file; already exists in vtkCoordinate.cxx.
Probably refactoring leftovers.

Change-Id: Ib2b935908e02c353dbed845786988e0fcb3d8f49
Removed unused VTK_FTTM_DEBUG_CD macro

It appears nowhere else in the codebase.

Change-Id: Ib7375958086e8b60c48895830bbd5268ba8a3488
Removed unused sameDirection macro

It appears nowhere else in the codebase. Found this dead code using -Wunused-macros.

Change-Id: I66bc2a2ebab4c3e69aa47b9808b57926e50163bc
Removed unused VTKQUATERNIONINTERPOLATOR_TOLERNCE macro

It appears nowhere else in the codebase. Found this dead code using -Wunused-macros.

Change-Id: I9e6f4ddbf2f1f00fc6c7ed5ca96bf1fed76038f1
Removed unused MAX_FACE_POINTS macro

It appears nowhere else in the codebase. Found this dead code using -Wunused-macros.

Change-Id: Iaa2fec28b6ccd1ae0f5f04746425a73c2dfd1973
Moved unused D2R and R2D macros into comment

They are only used in commented out code.
Moved into comment.
(Really this should use vtkMath anyway.)
Found this dead code using -Wunused-macros.

Change-Id: I4f4af4c6228455c4663ca8c603d4a053d36cff1b
Removed unused VTK_FREE_ARG macro

It appears nowhere else in the codebase.
Found this dead code using -Wunused-macros.

Change-Id: Ifbd9ab7954463d6b00e3a38ce464b4be5f74eb4d
Removed unused CONSTANT_DECL macro

It appears nowhere else in the codebase.
Found this dead code using -Wunused-macros.

There's an off chance, I suppose, that it
is expected by a system header, but I
don't think so.

Change-Id: Ib306770cd9c2aab05866f2bbdabb6f1aba2d435c
Removed unused vtkReebGraphMin macro

It appears nowhere else in the codebase.
Found this dead code using -Wunused-macros.

Change-Id: I1cbd97cd6905b5fd8dae878ef1a099522b9153e8
Removed unused FreeItem macro

It appears nowhere else in the codebase.
Looks like refactoring leftovers.
Found this dead code using -Wunused-macros.

Change-Id: I00f608f26f9eada928a6c910786a6fe218d3dd82
Removed unused macros

They appear nowhere else in the codebase.
Found this dead code using -Wunused-macros.

Change-Id: I1186e349eba2a5de8589191e33d1e225fd7c0e0e