ENH: Improve readability using vtkNew where possible

In addition to be shorter, it simply conveys the idea of locally
instantiating an object.

Indeed, vtkNew allow to instantiate an object on the heap and deleting it
when it goes out of scope if its reference count has not been incremented by
either assigning the object to a smart pointer or calling Register.

This commit continues the work initiated in commit r22539

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22896 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Fix unused variable warnings.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22897 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Update VTK external project to fix clang compilation issue

A new branch named "slicer-release-5.10_2014-01-20" has been pushed
to the Slicer VTK fork. It contains all the Slicer VTK specific commits
rebased on top of the HEAD of the vtk 5.10 release branch available
when this commit has done.

The difference between the v5.10.1 and the head of the vtk 5.10 release
branch are reported below.

$ git shortlog d3b66526624ba8e55addcddb0ec28c40982473ac...v5.10.1 --no-merges
David Cole (1):
      Rendering: Use AdjustWindowRect to ensure pixel perfect client size

Goodwin Lawlor (6):
      Backport using Tcl_GetErrorLine with Tcl8.6
      Let macro find TclTk version 8.6
      Update the TclTk internal files for 8.6
      Backport using Tcl_GetErrorLine with Tcl8.6
      Let macro find TclTk version 8.6
      Update the TclTk internal files for 8.6

Hans Johnson (1):
      COMP: Fix compiler error on clang c++11

Sean McBride (5):
      Conditionalize insane "#define inline" in libtiff
      COMP: fixed compilation error using llvm's libc++
      COMP: fixed llvm libc++ build error by removing invalid 'operator<<'
      Fixed compile error with llvm libcxx
      added missing #include, fixing compiler error with llvm's libc++

From: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22898 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Updating SimpleITK to 0.8rc1 and SWIG to 2.0.12

This version of SimpleITK addresses compilation issue with clang 5.0
on OSX with libc++. SimpleITK will now automatically be compiled with
"-std=c++11".

From: Bradley Lowekamp <blowekamp@mail.nih.gov>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22899 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: fixed improper importing a series of dicom images.

Fixes #3386

From: Kevin Wang <kevin.wang@rmp.uhn.ca>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22900 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: fixed qSlicerTransformModuleWidget::Invert method and qSlicerTransformModuleWidget test.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22901 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: Update tag for OpenIGTLinkIF in response to issue #3596.

From: Junichi Tokuda <tokuda@bwh.harvard.edu>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22902 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: get CLI module logic temp dir from application

Resetting the temporary directory at the application settings level
wasn't propagated to the CLI module logic. This change has the CLI
module logic getting the temporary directory from the application
as necessary.

Issue #3587



git-svn-id: http://svn.slicer.org/Slicer4/trunk@22903 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Remove now obsolete patching of BRAINSTools ExternalData module

Considering both projects now use the same version of the module, it is
not required anymore to copy it.
See the associated BRAINStools issue for more details: https://github.com/BRAINSia/BRAINSTools/issues/3

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22904 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Do not force BRAINSTools options to ensure they can overwritten

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22905 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: Extend 'Slicer_RemoteModule_Add' to accept list of cache options

When the VARS argument is set to a list of variable of the form:

  <name1>:<type1>=<value1> [<name2>:<type2>=<value2> [...]]

For more details, see inlined documentation.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22906 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Simplify how BRAINSTools project options are specified

Source repository, revision and associated CMake options are not
all set in a single location.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22907 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: update the versions of CompareVolumes and LandmarkRegistration

Improved plugin support for plastimatch (SlicerRT) and also improved
visualization via the RevealCursor and Rock modes.

Also fix the tests by selecting the module at the start of the
test (when running as a self test slicer starts with the default
module).

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22908 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: ExternalProject_Add_Source - Do not create build directory

Specify the 'BUILD_IN_SOURCE' argument to skip the creation of
unwanted build directory.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22909 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: Associated hierarchy node with ChildNode(Added/Removed)Event

It will now be easier to know which node has been added or removed
from a given hierarchy.

Still need to figure out which node to associated with the event
'ChildNodeRemovedEvent' invoked in function 'vtkMRMLHierarchyNode::RemoveHierarchyChildrenNodes'

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22910 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Update VTK project to re-add support for additional call data types

Following r22898, the version of VTK has been changed to use the
branch named "slicer-release-5.10_2014-01-20". When initially created,
it didn't include the improvement to the CallDataType support. This commit
fixes this by including the following to commit into the SlicerVTK fork:

$ git shortlog e7464be93..4e0ca68
Jean-Christophe Fillion-Robin (2):
      vtkPythonCommand - Add support for additional call data types
      vtkPythonCommand - Fix indent and variable names

Note that this commit have already been contributed and integrated
into VTK6.
See http://review.source.kitware.com/#/t/3966
and https://github.com/Kitware/VTK/commit/3a2a3ce595daab0bc2a9f858ef593905fd77a9ce

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22911 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: reset table range for label map volumes

Label map volumes need to be mapped to colors in the color
look up table. For editing, don't allow procedurally defined
color nodes to be chosen.

Issue #3586



git-svn-id: http://svn.slicer.org/Slicer4/trunk@22912 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: C++11 conformance for static initialization

error: non-constant-expression cannot be narrowed from type 'unsigned long' to 'IndexValueType' (aka 'long') in initializer list [-Wc++11-narrowing]
as identifiied by the clang compiler.

From: Hans Johnson <hans-johnson@uiowa.edu>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22913 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: C++11 compiler warning convert NULL to bool

warning: implicit conversion of NULL constant to 'bool' [-Wnull-conversion]

From: Hans Johnson <hans-johnson@uiowa.edu>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22914 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: C++11 convertsion of types

error: no viable conversion from 'DiffusionTensor3DExtended<float>' to 'DiffusionTensor3DExtended<double>'

By preferring initialization to assignment this error is avoided.

From: Hans Johnson <hans-johnson@uiowa.edu>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22915 3bd1e089-480b-0410-8dfb-8563597acbee