STYLE: Pep8 cleanup for generators

Change-Id: I3c4d787e84cfeb3ed35c49851e773247380ae624
BUG: Do not ENABLED_SHARED for GDCMIO

With shared libraries on windows and export specification with gcc,
memory leaking is being detected by valgrind. This should be
re-enabled when GDCM it self is enabled as shared with export
specifications.

Change-Id: Ib28f1e98cc66cd9cb4e28f935934290560fb99c9
DOC: CMake warning BRANWEB -> BRAINWEB.

Change-Id: Id0dbcb5c83807db7237a70b79a75b60ba8abd65b
BUG: Remove -Wno-unused-local-typedefs flag.

This flag was used to remove false positive warnings from GCC due to
ConceptChecking on the ITK tests.  However, it did not mask the false positive
warnings for projects that link against ITK.

Commit

  8329f68

fixed the warnings without requiring a flag.  Remove the compiler flag.

Change-Id: Ia1ec42564b2437921e5e0c8efe628aa3ae91f4d9
Suggested-by: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>
COMP: Fix "unused-local-typedefs" warnings

This commits fix warnings discovered while compiling Slicer. At the
exception of the change to "itkLinearInterpolateImageFunction.h", all
update consisted in removing line defining unused local typedefs.

Within "itkLinearInterpolateImageFunction.h", the "itkConceptChecking"
header has been included to make use of the itk::Concept::Detail
and get ride of warning for traits. Similar approach has been used in 8329f68.
DOC: Add break in brief description of Canny edge filter

Change-Id: I4f6e991953f7b125b34e8ffc426dd1a526336238
Merge topic 'BrainWebSpelling'

a73fa0b2 DOC: CMake warning BRANWEB -> BRAINWEB.
Merge topic 'GDCMNoENABLED_SHARED'

30aca620 BUG: Do not ENABLED_SHARED for GDCMIO
COMP: Fix default wrapping with all modules

My previous patch (http://review.source.kitware.com/#/c/15885/) broke the wrappings
when starting from an empty build dir, as the ITKGroup_ variables are only set when
ITK_BUILD_DEFAULT_MODULES was set once to OFF.

This will fix the errors on the dashboard.

Change-Id: I3b0d514caba3444ffeaa6837a915481877a9ca1d
Merge topic 'FixWrappingBuildALL'

e6f6e15a COMP: Fix default wrapping with all modules
Merge topic 'fix-unused-local-typedefs-warnings'

883d6dba COMP: Fix "unused-local-typedefs" warnings
COMP: Fix typo in ReflectiveImageRegionConstIterator.

Operate = needed to be operator=

Change-Id: I9c274438b369959c77551181e31a03e708e5f466
COMP: Fix "unused-local-typedefs" warning in LandmarkBasedTransformInitializer

This commit fixes the following Slicer warning:

// -------
In file included from /path/to/ITKv4/Modules/Core/Transform/include/itkLandmarkBasedTransformInitializer.h:188:0,
                 from /path/to/Slicer/Modules/CLI/FiducialRegistration/FiducialRegistration.cxx:7:
/path/to/ITKv4/Modules/Core/Transform/include/itkLandmarkBasedTransformInitializer.hxx: In member function ‘void itk::LandmarkBasedTransformInitializer<TTransform, TFixedImage, TMovingImage>::InternalInitializeTransform(itk::LandmarkBasedTransformInitializer<TTransform, TFixedImage, TMovingImage>::VersorRigid3DTransformType*)’:
/path/to/ITKv4/Modules/Core/Transform/include/itkLandmarkBasedTransformInitializer.hxx:252:65: warning: typedef ‘RotationCenterType’ locally defined but not used [-Wunused-local-typedefs]
   typedef typename VersorRigid3DTransformType::CenterType       RotationCenterType;
// -------

Change-Id: I8a632362eb73035df790a762362dd1a7665a4594
BUG: Add additional MetaDataObject explicit instantiation.

The MetaDataObject  of std::vector<std::vector<double> > > use by
Slicer's ResampleDTI module. This needed to be explicitly instantiated
and export to enable dynamic_casts to correctly work.

Additionally other related types have been added that may be used in
other applications.

Change-Id: Ic1f9d78bea6019bd03e2f079e0c828a18a9b2760
ENH: Use open() instead of file() for python 3 compatibility

file() is deprecated.
Use open() and with statement instead.

Change-Id: Ic983854cec41b0c3f6593c820bee184da2199060
ENH: Allow to use methods which pass std::string by reference from python

See ITK-3285.

Adding the following line for each argument in the .i files lets swig know what to do
with std::string when passed by reference:
%apply (std::string& INOUT) { std::string & variableName )

See the test for the syntax in python.

Change-Id: I1298327e2abb57e31ce0b23b24f1632fa46f653c
Merge topic 'DataObjectOfVectorVector'

9e820d93 BUG: Add additional MetaDataObject explicit instantiation.
COMP: Fix IOSTL Doxygen group and Windows shared build.

Change-Id: Idef2f73738dea7846ac78c303b14a744801cec80
Merge topic 'fix-unused-local-typedefs-warning-LandmarkBasedTransformInitializer'

b7f09768 COMP: Fix "unused-local-typedefs" warning in LandmarkBasedTransformInitializer
Merge topic 'ReflectImageRegionConstIteratorError'

ea5babf4 COMP: Fix typo in ReflectiveImageRegionConstIterator.