BUG: Coverity 1081062: Fix big parameter passed by value

The constructors for:
itk::Statistics::ImageToListSampleAdaptor::ConstIterator
itk::Statistics::ImageToListSampleAdaptor::Iterator
should have their first parameter passed by-reference, as it is an object.
The second parameter is just an unsigned int, so it can stay by-value.

Made the constructors for these 2 iterator classes use initializer lists.

Also cleaned up indentation style.

Change-Id: I0c990ac948565967cd8e12c37bb983f409597123
BUG: Add missing transform types to factory

Some transform types could not be read/written
because they were not registered on the TransformFactoryBase.

The names of these new default transforms were also
added to the test suite to verify that they are
present on every build.

Change-Id: I8234999f87a2c69070d0dd9269f97bbb1b4d3e8a
DOC: Improve documentation for StatisticsAlgorithm functions

Change-Id: Ibafadc912473c84f7bb54d7058b4a532366a0128
BUG: Coverity 1103200: Copy into fixed size buffer in GE5ImageIO

strtok is not guaranteed to be thread-safe and shouldn't be used anyway.

Change-Id: I7b71fbf047fc05745a4cefc6a67d92ff7ab97496
STYLE: Coverity 1080839: Dead default in switch in QuadEdgeMeshEulerOperatorJoinVertexTest

The code above the switch statement verifies that InputType is within
[0, 5]. The switch statement contains labels for all values [0, 5], so a
default statement is unnecessary.

This is not strictly a bug, as the behavior is unchanged.

Change-Id: I6e8b4494688161b9b4d45b6e4044375dd4552d2a
STYLE: Coverity 1080963: Dereference after null check in TreeIteratorBase

The instance variable m_Tree should never be NULL as a precondition of the
Add() method, and doesn't need to be explicitly checked for NULLness. A
TreeIteratorBase requires that m_Tree be set upon instantiation, and quite
sensibly, no other method bothers to verify that it is non-NULL.

Change-Id: I1208de1639b7c8b3261b3b55344152b75f32c587
BUG: Coverity 1081009: Missing break in switch in GiftiMeshIO

Change-Id: Iba5e0f028d7c1f28056e0a9be8c22a6aa66146fe
BUG: Coverity 1081422: Uninitialized pointer field in QuadEdgeMeshFrontBaseIterator

The initializer list is evaluated before the function body, so the
member variables can still be changed to non-NULL values later in the
function body.

Change-Id: I59c66de262b7cabb0bd45be2bee559a01ef21a5e
STYLE: Coverity 1081140: Dereference before null check

The NULLness of m_NiftiImage is checked immediately after it is set. None
of the subsequent statements or calls to nifti_read_subregion_image() or
nifti_image_load() (which both pass m_NiftiImage by-value) change the
value of m_NiftiImage. Thus, the removed NULLness check of m_NiftiImage
was superfluous.

Change-Id: I8d66cfe60c6e9adcf103f32ff2fa3e271b4b77b1
Merge topic 'cov_1080839'

0a08acb0 STYLE: Coverity 1080839: Dead default in switch in QuadEdgeMeshEulerOperatorJoinVertexTest
Merge topic 'cov_1081140'

e0b0423a STYLE: Coverity 1081140: Dereference before null check
Merge topic 'AddMissingTransforms'

3577a616 BUG: Add missing transform types to factory
Merge topic 'cov_1081422'

d0fd9d55 BUG: Coverity 1081422: Uninitialized pointer field in QuadEdgeMeshFrontBaseIterator
Merge topic 'cov_1103200'

bc6316f2 BUG: Coverity 1103200: Copy into fixed size buffer in GE5ImageIO
Merge topic 'Statistics_SampleBounds'

84ae681c DOC: Improve documentation for StatisticsAlgorithm functions
Merge topic 'cov_1081062'

aac0ab33 BUG: Coverity 1081062: Fix big parameter passed by value
Merge topic 'cov_1081009'

3b13d36b BUG: Coverity 1081009: Missing break in switch in GiftiMeshIO
Merge topic '1080963'

e19df2e4 STYLE: Coverity 1080963: Dereference after null check in TreeIteratorBase