Updates to the VTK coding conventions

I just merged some updates to the VTK coding conventions in this merge request. The main changes are:

  • Discourage default arguments in C++ function definitions.
  • Remove prohibitions due to VS 2013 because VS 2013 is no longer supported.
  • Update submodule discussion to mention the Viskores (VTK-m) submodule in relation to the no submodule policy.

The Coding Conventions are available for reference here.

GCC 4.8 is still supported? That is surprising.

Yes, some customers relied on it, but no longer. Once we move to C++17 support, the minimum gcc will be 8 something.

1 Like