Merge topic 'fix_itk-3153-SpatialObjectBoundingBoxes'

bf8de3f BUG: Fixes itkImageMaskSpatialObject part of issue ITK-3153
fa113e3 BUG: Fixes itkBoxSpatialObject part of issue ITK-3153
COMP: Conditional assert check warning unused var.

ITKv4/Modules/ThirdParty/VNL/src/vxl/core/vnl/vnl_diag_matrix.h:82:47: warning:
      unused parameter 'j' [-Wunused-parameter]
inline T& operator () (unsigned i, unsigned j) {

Change-Id: I8ca952cdbea58ab42879bfe362ed7a1ef505db1c
BUG: Avoid SimpleImageRegistration{Float,Double}Test output clobbering.

Fixes test failures on parallel builds.

Change-Id: I5ab4d8acf9cf7984185e81a2c0492465d83bdebc
BUG: Prevent MINC transform tests outputs from clobbering.

Use different file names to prevent test failures on systems that run the
tests in parallel.

Change-Id: I397e193399fdd1600414c9fb25788279b85075e3
Merge topic 'registration-double-float'

1ae8526 BUG: Avoid SimpleImageRegistration{Float,Double}Test output clobbering.
DOC: Fix a bug with SquaredEdgeLengthDecimationQuadEdgeMeshFilter

For an unknown reason, if the first line of the Doxygen comment block
for the SquaredEdgeLengthDecimationQuadEdgeMeshFilter::MeasureEdge and
::Relocate functions is blank, Doxygen fails in several significant ways.
This change fixes the content of the documentation and prevents warnings.

Change-Id: Ic66e494a411fc2bb3daa7acbb46d80d9cc56b2a6
DOC: Clean up comments in NiftiImageIO, for better Doxygen compatibility

Change-Id: I8441f6c31a07f7d17c9b89b014ff556bc542505a
STYLE: Remove unused typedefs from AlgorithmsPrintTests

These were left over from before AlgorithmsPrintTest was split into
separate files.

Change-Id: I04865517dfb17585b6975f25408d16642109fbe0
BUG: Fix improper usage of VoronoiSegmentationRGBImageFilter

VoronoiSegmentationRGBImageFilter requires a PixelType with 3 components.
Fixing this also resolves a compiler warning.

Stronger concept checking should also be added in the future to ensure
this usage: bug ITK-3231.

Change-Id: I0d624c8d4be9f3bedc962a5027714e90e54f693d
Merge topic 'Doxygen-Warning'

7a238de DOC: Fix a bug with SquaredEdgeLengthDecimationQuadEdgeMeshFilter
Merge topic 'NiftiComments'

5e239fc DOC: Clean up comments in NiftiImageIO, for better Doxygen compatibility
Merge topic 'AlgorithmsPrintTest'

194f2f0 BUG: Fix improper usage of VoronoiSegmentationRGBImageFilter
18ab102 STYLE: Remove unused typedefs from AlgorithmsPrintTests
Merge topic 'VXL_AssertWarning'

e204f99 COMP: Conditional assert check warning unused var.
Merge topic 'minc-transform-outputfile'

1929776 BUG: Prevent MINC transform tests outputs from clobbering.
ENH: Move OpenFileForReading/Writing from StreamingImageIO to parent class

The methods OpenFileForReading and OpenFileForWriting are useful for
nearly all ImageIO classes, not just StreamingImageIO.

Change-Id: I3bbe9155f2e052b4d5191758b2b09a242437077f
ENH: Improve OpenFileForReading/Writing logic and documentation

All changes are backwards-compatible.

Updated parameters:
* Renamed the first parameter to be more clear.
* Changed the second parameter to a std::string to provide utility and
prevent unnecessary type conversions.
* Made OpenFileForWriting's truncate parameter optional, with a default
value of true.
* Added an optional parameter for ASCII mode.

Improved documentation for both methods.

Made minor improvements to both implementations, and fixed a bug where
non-existent files could fail to open in read+write mode.

Change-Id: Id036ce9c504a14c857be77e7bad487e9a5c45d52
COMP: SimpleITK linkage failure

When building with clang on 10.9 where ITKv4 is a shared library,
a linking error occured for unfound symbols: NumericTraits< OutputPixelType >::Zero
but using the NumericTraits< OutputPixelType >::ZeroValue()
allows linkage to occur properly.

Change-Id: I496a41c102960b60c461adefdccc373b39dabb03
Merge topic 'FixZeroValue'

9ed7059 COMP: SimpleITK linkage failure
BUG: Prevent ObjectFactoryBase from possibly throwing an exception

Coverity produces over 250 errors due to the fact that
ObjectFactoryBase::RegisterInternal() has the structural capability to
throw an uncaught exception, if m_RegisteredFactories is non-empty.

However, as currently implemented, when ObjectFactoryBase::
RegisterInternal() is called, it is logically impossible for
m_RegisteredFactories to be non-empty. Verifying this precondition as a
check against future bugs is still good idea, but errors will now only be
produced in debug mode. In all build configurations, m_RegisteredFactories
will be cleared (which should be a small, constant-time overhead), to
guarantee consistent behavior.

Change-Id: Ia5890a323a7e8d5ed6b44195158a7a05f2c5df3e
STYLE: Make internal-use CMake variables lowercase in KWStyle.cmake

This better follows CMake style conventions.

Change-Id: I40a4a71e4c0d11ab0c2e12ee8ef34734e9f439af