[vtk-developers] Incorrect indentation by vtk_reindent_code.py

[David Gobbi]

On Mon, Jul 4, 2016 at 3:41 PM, Sean McBride <sean at rogue-research.com>
wrote:

>
> VTK's Whitesmiths style isn't supported by modern tools, and once we get
> away from it we can do things like autoformatting,


Yup, this nicely summarizes the original reason for the proposal.

git hooks that reject incorrect formatting, dashboard check of formatting,
> etc.


We could do this regardless of what style we choose :)

My python script already has a "--test" option.  Checking that braces are
present and that conditional blocks are correctly indented is easy.  We
could run this check alongside the current HeaderTesting test (but unlike
HeaderTesting, it'll run on .cxx files, too).

 - David
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20160704/d7b7719a/attachment.html>