<div dir="ltr">+1 to Brad's suggestion,</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 9, 2015 at 3:41 PM, Brad King <span dir="ltr"><<a href="mailto:brad.king@kitware.com" target="_blank">brad.king@kitware.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 9/9/2015 9:53 AM, Shawn Waldon wrote:<br>
> It would be nice to have a format that doesn't require twiddling<br>
> with the editor(s) so much.<br>
<br>
</span>Yes.<br>
<span class=""><br>
> Having said that, I prefer that we do what<br>
> Brad suggested and use clang-format.<br>
<br>
</span>After further thought I think we should delay consideration of<br>
clang-format.  Let's not allow it to get in the way of Berk's<br>
original proposal.  It has a lot of +1s.<br>
<span class=""><br>
On 9/9/2015 9:22 AM, David Lonie wrote:<br>
>   if (...) {<br>
>     ...<br>
>   }<br>
>   else {<br>
>     ...<br>
>   }<br>
</span>[snip]<br>
<span class="">On 9/9/2015 9:43 AM, Ben Boeckel wrote:<br>
> I've preferred using "} else {"<br>
<br>
</span>I also like<br>
<span class=""><br>
  if (...) {<br>
    ...<br>
  } else {<br>
    ...<br>
  }<br>
<br>
</span>and it seems to be widely used.  The placement of the opening<br>
curly on the block opening line or on its own line does not<br>
affect the indentation of the code inside the curly braces,<br>
so this choice can be made locally rather than mandated<br>
globally.<br>
<br>
--------------------------------------------------------------------<br>
<br>
I propose that we start with a sweeping change implementing<br>
Berk's original proposal of just moving the curly braces up<br>
one level of indentation:<br>
<span class=""><br>
  if (...)<br>
  {<br>
    ...<br>
  }<br>
  else<br>
  {<br>
    ...<br>
  }<br>
<br>
</span>Then in future edits and/or new code we can allow the opening<br>
"{" to be either on its own line or on the block opening line as<br>
preferred by individuals.  Changes to curly brace placement can<br>
be made incrementally as code is edited to avoid blame poisoning.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Brad<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<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></div>