DOC: Better description of what the code is doing.

Improved the code organization and variables names in order to better convey
the actions that the code is taken. This was driven by a warning reported by
a Coverity scan with text:

  1103094 Out-of-bounds access
  Memory not owned by this buffer will be accessed, causing memory corruption
  or incorrect computations.

  In

      itk::​Functor::​ScalarToRGBPixelFunctor<unsigned char>::
      ​operator ()(unsigned char const&):

  Access of memory past the end of a memory buffer

The scan warns about the risk of using out of bounds elements due to the use
of an array/pointer to navigate through the bytes of the scalar class. This
is however, the correct intent of the code, and bounds are checked properly
using the sizeof() the scalar type.

Change-Id: I1884ca33f5a7eea8bb28b043f8bcef4bff129805
COMP: Destination buffer too small

CID 1103098 (#1 of 1): Buffer not null terminated (BUFFER_SIZE_WARNING)
buffer_size_warning: Calling strncpy with a maximum size argument of
2049 bytes on destination array "curImage->filename" of size 2049
bytes might leave the destination string unterminated.

Change-Id: I2ed96404b9b2374a84406d6ea3d46b70006ae6a5
COMP: Destination buffer too small

CID 1103199 (#1 of 4): Destination buffer too small (STRING_OVERFLOW)
string_overflow: You might overrun the 35 byte destination string
"hdr->hospital" by writing 2049 bytes from "tmpStr"

CID 1103199 (#2 of 4): Destination buffer too small (STRING_OVERFLOW)
string_overflow: You might overrun the 32 byte destination string
"hdr->scanId" by writing 2049 bytes from "tmpStr".

CID 1103199 (#3 of 4): Destination buffer too small (STRING_OVERFLOW)
string_overflow: You might overrun the 32 byte destination string
"hdr->patientId" by writing 2049 bytes from "tmpStr".

CID 1103199 (#4 of 4): Destination buffer too small (STRING_OVERFLOW)
string_overflow: You might overrun the 32 byte destination string
"hdr->date" by writing 2049 bytes from "tmpStr".

In all cases, strcpy was replaced with strncpy.

Change-Id: I9955d3c34970b89dfddaa233dfbcb8943166f1be
BUG: Resource leak

CID 1106766: Resource leak (RESOURCE_LEAK)
overwrite_var: Overwriting "cinfo" in "cinfo =
opj_create_compress(CODEC_JPT)" leaks the storage that "cinfo" points
to.

CID 1106766: Resource leak (RESOURCE_LEAK)
overwrite_var: Overwriting "cinfo" in "cinfo =
opj_create_compress(CODEC_JP2)" leaks the storage that "cinfo" points
to.

Change-Id: I030a5fa5455a64251423399714c9e0796c799242
COMP: Resource leak

CID 1103190: Resource leak (RESOURCE_LEAK)
leaked_storage: Variable "mySceneChildren" going out of scope leaks
the storage it points to.

Added a delete mySceneChidren before each return EXIT_FAILURE

Change-Id: I5a0a46dbe1034da65727dac69b9b9244371c689a
COMP: Big parameter passed by value

CID 1125947 Copying large values is inefficient, consider passing by
reference; size thresholds for detection can be adjusted.

In ComputeAreaError<…>(T1, unsigned int): A large function call
parameter or exception catch statement is passed by value

Change-Id: I813f55c9531d14d208dc5bc3b33d425d3121a92a
Merge topic 'Coverity_itkJPEG2000ImageIO'

387b74d BUG: Resource leak
Merge topic 'Coverity_itkGE4ImageIO'

3d5ddd0 COMP: Destination buffer too small
Merge topic 'coverity_cid_1103094_out_of_bounds_access'

19e43d4 DOC: Better description of what the code is doing.
Merge topic 'Coverity_itkGE5ImageIO'

550f6ff COMP: Destination buffer too small
Merge topic 'Coverity_itkReadWriteSpatialObjectTest'

358e443 COMP: Resource leak
Merge topic 'Coverity_itkFlatStructuringElementTest'

f68a325 COMP: Big parameter passed by value