BUG: Border visibility from underlying vtkScalarBar actor

Get the correct border visibility for the scalar bar
representation from the underlying scalar bar actor instead of
setting a default property. This change fixes issue 14646.

Change-Id: I7c11033475ebe8e3009bb9c3a0ebc745dbc63c23
python: fix leaks when setting constants in wrappers

PyDict_SetItemString does not steal a reference to the value; the caller
is still in charge of its reference if it is set.

See the example in the PyDict_Next example[1] in the Python docs.

[1]https://docs.python.org/2/c-api/dict.html

Change-Id: Ib4927db09ae3f32161b1af4434a8965bb590987d
Fix a crash when deleting vtk-python objects.

This fixes a bug that has been present since commit 269cbea1 on Oct 16,
2013 where deleting an object that had a DeleteEvent observer would cause
a crash (the crash was only verified to occur on OS X).

Change-Id: I89d7a9a6eff9ab11fdcd7291a42f47b9a8017091
STLWriter: Raise an error for non-triangles

The STL format only supports triangles and previously, it just truncated
polygons silently. Cause an error to occur when this happens so that
users know.

Change-Id: I9c2f96d887074943130a8442d6796cb0c2ce1bbf
STLReader: Close the file when an error occurs

Change-Id: I38a189e6af609836a846d892050440e69473a9b7
STLReader: Fix error signaling when reading files

The RequestData return value of 0 means an error and make the actual
reader implementations return a boolean.

Change-Id: I3caf27782fb34e5e5474f40ef8d466bda61cd12c
BUG #14599: Handle double values properly.

Plots used to rendering transfer functions failed to work correctly when
the values/ranges they were presenting were in the double realm. This
was because these classes didn't handle vtkPlot::ShiftScale at all. This
fixes that issue.

Change-Id: I08c487c6e735b5a7055daff7e537e97036449e96
BUG0013829: vtkAssembly::GetBounds adds bounds which are uninitialized

If an assembly part's bounds have not been initialized, the assembly
still accumulates that part's bounds. This results in invalid bounds.

Added a test based on the solution proposed by the bug reporter.

Thanks to Joo and Serge for the patch.

See: http://www.vtk.org/Bug/view.php?id=13829 for details.

Change-Id: If4b2f93d28da77072306997cde84b8f6f35b85fe
Fixing overflow issue that appeared with GCC compilers.

Thanks to Michael Stephens for the report and fix.

Change-Id: Ibe68819a81b011d3e3b0d8fefc2bc122834e8602
Added "sticky" axes mode to vtkCubeAxesActor

This change adds a mode to vtkCubeAxesActor that keeps the axes in the
viewport by changing the boundaries of the axes to keep them all
within the viewport. It works by computing the largest sphere that
will fit in the view frustum at the depth of the original bounding
box's center, then computes a cube that fits within that sphere.  The
final bounding box is the intersection of the original bounding box
and the cube that fits within the sphere. If this intersection is
empty, the bounds are invalidated so that no axes are displayed.

The sticky axes mode also has a centering option that controls whether
the truncated bounding box stays centered in the viewport or follows
the center of the original bounding box along the longer of the two
viewport dimensions. Whether this option is enabled or not, the axes
are restricted to being within the viewport.

Change-Id: I6dc3cd4127d2e4da8f2580fd2242ffa51270629a
BUG: Add a missing include for VS 2013

Change-Id: I4f384d4860e6d333c4027e8b8617f7923ab7141e
Fix numerical precision problem in vtkCellLocator::IntersectWithLine

Change-Id: I07e71146fbd6f8e68661cdc941b9a0c473613144
Add default value for argument

Change-Id: I89511c45e4c4c0f585d9c04eaca0131066db975b
Set origin and spacing in vtkSurfaceReconstructionFilter. Ref: Mantis 0002826.

Change-Id: Id2c65e4f45d40a7da90644ad14a8d7a7feb84338
Fixed bug causing plane to disappear on boundary.

When the plane was moved to one of the sides of the bounding region, and
was parallel to the bounding region, it sometimes disappeared. To fix this
the origin of the plane is clamped to slightly within the bounding region.

Change-Id: Ie4c22df9faf9d6dc739dd542f07df136f0784e9d
Fix warning about trailing comma in enum list.

Change-Id: I82f4b76f30f7b7b57c6050727a5927516791a2d4
Clang: commas at the end of enumerator list are a C++11 extension
[-Wc++11-extensions]
Fix clang analyzer warning about passing null to strcmp

The pointer was being checked for null, but only logged; now return too.

Change-Id: Id9af0f968fc4fd626d9eec79af1ee22aebdc6cc6
Fixed clang analyzer warning about passing null to memcpy

In an error code path, vtkErrorMacro is used but the 'return' was omitted (every other vtkErrorMacro is paired with return), execution would then continue and pass null to memcpy.

Also removed unrelated useless test for null before using delete.

Change-Id: I9d63fae00080c2fca5974442d8bd2f936fcf0790
Fixed clang analyzer warning about reading uninitialized memory

If neither 'DMapData' nor 'MapData' get set, 'results' is read uninitialized; added 'else' case and bail out instead.

Change-Id: Ifa35e7ff76a1d3d9e74d446f1b9f3c4b42e3bc9a
Fixed clang analyzer waring about a dead store.

Useless store to local variable right before a return.

Change-Id: I0bd514edec4c7e7ac5409f68c50dae6bdee5a056