* 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>

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

Joachim Pouderoux authored 9 years ago and Kitware Robot committed 9 years ago

bf1cd7e2 Change iterator from postincrement to preincrement
5c97e44a Move variable declaration inside loop to limit its scope
5e1fc1bf Fix indentation and restore empty line
761a643e Add field data to vtkUniformGrid to save GDALDataset NoDataValue

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !344

Fix uninitialized memory reported by valgrind.

23dff2bd introduced a bug where it didn't copy the point data correctly
for all the layers. Fixed that.

See merge request !438

23dff2bd introduced a bug where it didn't copy the point data correctly
for all the layers. Fixed that.

Berk Geveci authored 9 years ago and Kitware Robot committed 9 years ago

f4126e3a Fixed bug with loading 2D or ASCII datasets in parallel.

Acked-by: Kitware Robot <kwrobot@kitware.com>
Reviewed-by: Utkarsh Ayachit <utkarsh.ayachit@kitware.com>
Merge-request: !434

David Lonie authored 9 years ago and Kitware Robot committed 9 years ago

1118440f Make ScalarBarRepresentation orientation set-able.

Acked-by: Kitware Robot <kwrobot@kitware.com>
Reviewed-by: Utkarsh Ayachit <utkarsh.ayachit@kitware.com>
Merge-request: !423

Had to move the code that broadcasted settings to ranks > 1
because the settings are needed to make a decision about
2D / ASCII parallel code path.

Berk Geveci authored 9 years ago and Kitware Robot committed 9 years ago

d91ecf98 BUG: Was using the wrong extent in computation.
005e70d3 Fixed warnings.
5f15c59d Cleaned up error handling in plot3d reader.

Acked-by: Kitware Robot <kwrobot@kitware.com>
Reviewed-by: Utkarsh Ayachit <utkarsh.ayachit@kitware.com>
Merge-request: !433

- Minor error handling cleanup.
- Made sure that reading error on rank 0 causes all
ranks to report error.

Berk Geveci authored 9 years ago and Kitware Robot committed 9 years ago

557ec619 Error checking: check status of collective call.
d9555377 Fix typo in call to superclass method.

Acked-by: Kitware Robot <kwrobot@kitware.com>
Reviewed-by: Berk Geveci <berk.geveci@kitware.com>
Merge-request: !432

Ensure we check status for all collective calls correctly.

Copy/paste error!