This appears to have been broken by !7193 (SHA 889ed965)
which consumed double-clicks further upstream but did not
expose them again to interactor styles.

These were all found by clang-tidy's readability-inconsistent-declaration-parameter-name option.  It also auto fixes them by changing the declaration to match the definition.  But often that choice is worse and often comments need fixing too, so I carefully chose only some of its fixes.

Used this script then manually reverted changes to ThirdParty and the like:

find . -iname "*.cxx" -exec sed -i '' s:^//-----------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//----------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//---------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//--------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//-------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//-----------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//----------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//---------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//--------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//-------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//--------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//---------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//----------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//-----------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//------------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//-------------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//--------------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//---------------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +

find . -iname "*.cxx" -exec sed -i '' s:^//\ -----------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ ----------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ ---------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ --------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ -------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ ------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ -----------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ ----------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ ---------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ --------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ -------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ --------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ ---------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ ----------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ -----------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ ------------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ -------------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ --------------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +
find . -iname "*.cxx" -exec sed -i '' s:^//\ ---------------------------------------------------------------------------------------$:\/\/\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\-:g {} +

This will allow one day making those getters return const

This check replaces default bodies of special member functions with
= default;. The explicitly defaulted function declarations enable more
opportunities in optimization, because the compiler might treat
explicitly defaulted functions as trivial.

Additionally, the C++11 use of = default more clearly expreses the
intent for the special member functions.

The check flags insertions to an STL-style container done by calling the
push_back method with an explicitly-constructed temporary of the container
element type. In this case, the corresponding emplace_back method results in
less verbose and potentially more efficient code.

These interactors are based on their native
UI but do not have any requirement on OpenGL.
Move them into this new directory so that Vulkan
or other toolkits can make use of them.

While at it remove the use of Xt as it is no longer
really used and instead simplify the code and go directly
to Xlib.

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

Run the `clang-format.bash` script to update all our C and C++ code to a
new style defined by `.clang-format`.  Use `clang-format` version 8.0.

* If you reached this commit for a line in `git blame`, re-run the blame
  operation starting at the parent of this commit to see older history
  for the content.

* See the parent commit for instructions to rebase a change across this
  style transition commit.

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

clang-format treats them as declarations and formats accordingly.

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

clang-format takes the lack of a semicolon as if it were an attribute or
some other "chainable" call leading to bogus indentation. By using
semicolons on all macro calls, the formatting comes out the way it
should.

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

Includes a few changes related to extraneous whitespace fixes and
deduplication of headers as well.

See the vtkCellArray.h class docs for details.

This also makes the `vtkQtTableRepresentation.h` header get installed as
well.

(cherry picked from commit 3cd6b857)

This also makes the `vtkQtTableRepresentation.h` header get installed as
well.

The dynamic analysis tests have uncovered a number of floating point
exceptions, mainly divide by zero. This MR enables floating point
exceptions if tests are built with a debug build type and the
compiler is a gnu compiler.

Safety guards have been added to classes that exhibited exceptions.

TestAdjustLowerBoundForLogPlot.cxx - Tried to take log10 of negative number

TestlinePlotDouble - inf and nan used in test. Disable exceptions

vtkChart.cxx - guards to protect against zero divide

vtkChartBox.cxx - guards to protect against zero divide

vtkInteractiveArea.cxx - trying to take log10 of negative number

vtkPiecewisePointHandleItem.cxx - protect against degenerate lines

vtkAMRSliceFilter.cxx - protect against zero length vector

vtkLagrangianBasicIntegrationModel.cxx - guards to protect against zero divide

UnitTestMultiThreshold - disable exceptions for one test that uses Nan()

vtkIntersectionPolyDataFilter.cxx - guard acos range

vtkPCAStatistics.cxx - guards to protect against zero divide

vtkCubeAxesActor.cxx - guard against log10(0.0)

vtkColorTransferFunction.cxx - guard to protect against zero divide

TestGPURayCastIsosurface.cxx - disable exceptions, possible driver issue

vtkDendrogramItem.cxx - check for degenrate line

vtkDendrogramItem.h - comparison doesn't meet irreflexive requirements

Marked as legacy in 8.2.

These new baselines are largely for new text rendering from the newer
freetype.

Same speed but much better end result than the old
transparency code.

Also fix a few classes that were not passing properties
to their sub props.

Fix medical example to make opaque objects opaque.

Found via `codespell`

Ken Martin authored 6 years ago and Brad King committed 6 years ago

make image difference symmetric to catch some
failures that were slipping through. This exposed
a number of valid images that were no longer
correct and needed to be replaced.

Kitware Robot authored 6 years ago and Brad King committed 6 years ago

Populate a script with `map_<md5>=<sha512>` variable settings and
then run the following to convert the content link files:

    git ls-files -- '*.md5' | while read f; do
      md5="$(cat "$f")"
      sha512="$(eval echo \${map_$md5})"
      rm "$f"
      echo $sha512 > ${f%.md5}.sha512
    done

Found via `codespell` and `grep`

154 occurances of ;;\n

This required some clean up, because the automatic changes borked a couple of files, but mostly resulted in stupid whitespacing. A few find/replace fixed things up.

There are now some duplicate semis because it did changes like:

  ~AMRIndexIterator() override{};
to
  ~AMRIndexIterator() override= default;;

Note there was a pointless semi before, which clang-tidy didn’t expect / account for.

My next commit will remove them.