[vtk-developers] PROPOSAL: Changing VTK's indentation style

Cory Quammen cory.quammen at kitware.com
Wed Sep 9 10:08:25 EDT 2015


+1

Everyone I know has commented on the oddity of the current brace style. I'm
used to it now, but it won't take much to adjust to the proposed style.

I think I prefer braces on separate lines to keep conditional keywords
aligned vertically.

On Wed, Sep 9, 2015 at 9:57 AM, Sean McBride <sean at rogue-research.com>
wrote:

> On Wed, 9 Sep 2015 09:08:53 -0400, Berk Geveci said:
>
> >What would you think of changing VTK's indentation style from:
> >
> >if ()
> >  {
> >  // ..
> >  }
> >
> >to
> >
> >if ()
> >{
> >  // ...
> >}
>
> +100  :)  I rather hate the current style. :)
>
> I've used clang-format and uncrustify too, and vote that whatever we
> choose we should be sure it works with these types of tools.
>
> Personally, I prefer the braces on their own lines, but putting the
> opening brace at the end of the line is fine too.
>
> Agree with Ben that we should always use braces, even for one-line 'ifs'.
>
> Cheers,
>
> --
> ____________________________________________________________
> Sean McBride, B. Eng                 sean at rogue-research.com
> Rogue Research                        www.rogue-research.com
> Mac Software Developer              Montréal, Québec, Canada
>
>
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Search the list archives at: http://markmail.org/search/?q=vtk-developers
>
> Follow this link to subscribe/unsubscribe:
> http://public.kitware.com/mailman/listinfo/vtk-developers
>
>


-- 
Cory Quammen
R&D Engineer
Kitware, Inc.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20150909/b6fefed8/attachment-0001.html>


More information about the vtk-developers mailing list