Suppressed errors about using possibily uninitialized values by giving a default value
Fixed warnings about unused local variables
fixed "warning: integer constant not in range of enumerated type 'mitype_t' [-Wassign-enum]"

Indeed the mitype_t enum has no 0 value.  I'm guessing MI_TYPE_UNKNOWN (-1) is better, but not sure.
removed dead private/static function transform_or_invert_point()

fixes warning about unused function.
Added newline at end of file, fixing warning.
Renamed variable that was shadowing another outer variable.

Renamed inner variable from 'i' to 'j'.
Fixed return value of copy_auxiliary_data_from_minc2_file() function.

The function's comments say "RETURNS : VIO_OK or VIO_ERROR" but it would sometimes return -1, which I'm assuming should be VIO_ERROR.

Fixes compiler warning about -1 not being one of the VIO_Status enum values.  Indeed, that enum has no negative values.

This could be dangerous if clients of the function are depending on sometimes getting -1 though.  On the other hand, the current situation is also dangerous if clients are depending on the docs being right!
Fixed/suppressed a bunch of warnings about unused variables

Most were genuinely unused, probably leftover from previous refactoring.
A few are unused but only because their use is commented out, so I commented out the declarations too.
fixed classic off-by-one error in miget_attr_values()

For strings, the code is always forcing null termination
but was writing just after the last byte of the buffer.

Caught by clang address sanitizer, which detected the buffer overrun
in the "minc2-grpattr-test" test.

Added additional test too, that exercises truncation by 1 byte.
Merge remote-tracking branch 'upstream/develop' into develop

Conflicts:
	libsrc2/grpattr.c
	testdir/minc2-datatype-test.c
	volume_io/Volumes/output_mnc2.c
removed some copy-pasted comment that is incorrect in these contexts
added another test for miget_attr_values(), to catch off-by-one errors
- cleanup #if 0'ed code
- restored needed variables in #if 0 block
Added CTestConfig.cmake file for dashboard support
Removed unused variable, warned by -Wall
Suppress used parameter warnings for main's argc and argv.

Obviously, this doesn't add a lot of value per se, but the warning
are part of -Wextra, and -Wextra is a reasonable level of warnings
that should be achievable.
Suppressed unused parameter warning

The function's arguments are mandated by HDF5
so this is likely genuinely meant to be unused.
Fixed some -Wsign-compare warnings

Two types:
1) Just added a cast. The signed value was confirmed non-negative a few lines earlier, so cast it to unsigned.
2) changed a for loop index variable from unsigned to signed by adding a new 'j' index variable different from the 'i' one.