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
PERF: Prevent FindKWStyle.cmake from being called multiple times

Every run of FindKWStyle.cmake has the additional cost of executing
kwstyle to get its version. This change prevents FindKWStyle.cmake from
being called twice, except on a fresh configuration.

Change-Id: Ied7949bad56363f73a8e89485207536bcea097ae
BUG: Fix FindKWStyle crash when kwstyle returns empty version info

One of the dashboard machines has a version of KWStyle which returns
only an empty string or whitespace in response to "kwstyle -version". The
cause of this is unknown, but this commit will prevent FindKWStyle.cmake
from crashing in this case.

Change-Id: I093e17642d5a2ed26bb1e4be1960ce66664da787
Merge topic 'ObjectFactoryException'

302d53c BUG: Prevent ObjectFactoryBase from possibly throwing an exception