* freetype-32bit-error-release:
  vtkFreeTypeTools: support VTK_USE_64BIT_IDS=OFF

* gl/danlipsa/tiff-writer-loop-15654:
  BUG #15654: Saving a wavelet as TIFF loops.

Saving a wavelet data in ParaView of size (-300, 300)^3 loops.
The progress bar goes from 0 to 100 several times.

* gl/ben.boeckel/msvc2015-support:
  vtkWin32OpenGLRenderWindow: static_cast to BYTE
  vtkWin32OpenGLRenderWindow: fix some whitespace
  vtkFreeTypeTools: use vtkIdType for cache ids
  msvc2015: fix narrowing warnings
  hdf5: fix long long format detection
  hdf5: detect %I64d specifier
  vtkEnSightReader: refactor a call
  vtkhdf5: don't expect timezone with VS2015
  vtktiff: look for search.h
  libxml2: fix snprintf workaround

MSVC2015 warns about narrowing.

* gl/brad.king/revert-RemoveBoostForwardDeclarations:
  Revert "Remove forward boosts declarations"

This reverts commit 50d2a986.
The vtkInfovisBoostGraphAlgorithms module uses these forward
declarations and friendship.

* gl/tjcorona/skip-installing-private-headers:
  Added a means to prevent private headers from being installed.

* gl/tjcorona/issue_0015533:
  Force a lookup table reset after vtkAbstractArray::Reset()

* gl/tjcorona/issue_0015390:
  Removed double delete in vtkXdmfWriter destructor.

* gl/tjcorona/issue_0015300:
  Prevent str.begin() from being dereferenced if str is empty.

* gl/tjcorona/issue_0015525:
  XML data reader now throws an error when reading arrays with duplicate names.

This fix addresses bug reports 0014698 and 0014700. Header files that were
named identically to source files were installed. The added flag allows for
a source file to be flagged with SKIP_HEADER_INSTALL, which prevents its
associated header file from being installed (thanks Ben Boeckel!).

This fix is in reference to bug report 0015533. The method Reset() in
vtkAbstractArray was modified to flag a change in data, resulting in the
correct behavior for LookupValue() after Reset() was called. Also, the method
DeepCopy() was changed to only copy valid elements (thanks David Lonie!).

This fix is in reference to bug report 0015525. The reader would not read
arrays with duplicate names, but would count them when determining the number
of components, resulting in a segmentation fault. The correct behavior, which
prevents multiple arrays with the same name, is now in place.

This fix is in reference to bug report 0015390. DomainMemoryHandler was being
deleted twice. It is now deleted once, and is subsequently set to NULL.

This fix is in reference to bug report 0015300. In
vtkXmlWriter::vtkXMLWriteAsciiValue(ostream& os, const vtkStdString& str), the
first char in the string was being accessed before checking the string's
length. This caused problems with empty strings.

* gl/ben.boeckel/FixLinuxCompilerExtrasWarningsAsErrors:
  linking: default to not making warnings fatal errors

Joachim Pouderoux authored 9 years ago and Ben Boeckel committed 9 years ago

Toolchains tend to add new warnings as time goes on, so in the spirit of
keeping VTK as future-proof as possible, don't turn them into warnings
by default. Testing machines and developers can certainly enable it
though.

    http://public.kitware.com/pipermail/vtk-developers/2015-July/032233.html

* gl/seanm/SimpleCocoaVTK6.3:
  Update SimpleCocoaVTK Xcode project to work with VTK 6.3

* gl/jpouderoux/FixDegeneratedPolygonCrash:
  Fix crash in vtkPolygon::ParametrizePolygon for invalid polygons.

* gl/cory-quammen/vtkColorTransferFunction_crash:
  Fix crash where vtkColorTransferFunction has 0 nodes

* gl/dgobbi/contour-triangulator-normal:
  Fix normal computation in vtkContourTriangulator.

If the polygon has less that 3 points, the function call leads to
a crash.
Add a simple test on the number of point to avoid that.

* gl/demarle/fix-value-painter:
  fix a bug where leftover color state would break value painter

* gl/johan-andruejol/RemoveBoostForwardDeclarations:
  Remove forward boosts declarations

Thanks to Jean-Christophe Fillion-Robin for pointing this out.

See

https://github.com/Slicer/Slicer/commit/f59a689a0f6d2912538ed72dad16efc70601f0ed#commitcomment-9407867



for reference.

A basic test for vtkColorTransferFunction is added that crashes
prior to this patch and succeeds with this patch.

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

The ID is the integer representation of a pointer, so use a
pointer-sized type.

The code does a loop itself, so just run it once and use its output as
the result.

In fact, this code would always return "ll" because if the file had a
right answer, it would return 0 for it, but then think it was "ll" that
passed, not one of the formats in the file.

MSVC2015 has long long and uses I64 as the modifier. Other modifiers
cause the standard library to throw up runtime errors and causes the
test to fail.

MSVC2015 ICE's without this in the optimizer somewhere. It's better this
way anyways.

It went away?

It is asked for in tiffiop.h, but never defined.

MSVC2015 is a team player!

Ben Boeckel authored 9 years ago and Kitware Robot committed 9 years ago

2166c067 FiltersStatisticsGnuR: fix R_NO_REMAP juggling

Acked-by: Kitware Robot <kwrobot@kitware.com>
Reviewed-by: Brad King <brad.king@kitware.com>
Merge-request: !437