<div dir="ltr">+1<div><br></div><div>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.</div><div><br></div><div>I think I prefer braces on separate lines to keep conditional keywords aligned vertically.<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 9, 2015 at 9:57 AM, Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com" target="_blank">sean@rogue-research.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, 9 Sep 2015 09:08:53 -0400, Berk Geveci said:<br>
<br>
>What would you think of changing VTK's indentation style from:<br>
><br>
>if ()<br>
>  {<br>
>  // ..<br>
>  }<br>
><br>
>to<br>
><br>
>if ()<br>
>{<br>
>  // ...<br>
>}<br>
<br>
</span>+100  :)  I rather hate the current style. :)<br>
<br>
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.<br>
<br>
Personally, I prefer the braces on their own lines, but putting the opening brace at the end of the line is fine too.<br>
<br>
Agree with Ben that we should always use braces, even for one-line 'ifs'.<br>
<br>
Cheers,<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
____________________________________________________________<br>
Sean McBride, B. Eng                 <a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a><br>
Rogue Research                        <a href="http://www.rogue-research.com" rel="noreferrer" target="_blank">www.rogue-research.com</a><br>
Mac Software Developer              Montréal, Québec, Canada<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">http://public.kitware.com/mailman/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Cory Quammen<br>R&D Engineer<br>Kitware, Inc.</div>
</div>