<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 21, 2013 at 7:05 PM, Karl Rupp <span dir="ltr"><<a href="mailto:rupp@mcs.anl.gov" target="_blank">rupp@mcs.anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
     Things like<br>
<br>
     if (y < 12) {ierr = Something(); CHKERRQ(ierr);}<br>
<br>
don't match the standard but …. am I being too picky? I believe the PETSc make uncrustify rule would move that to separate lines.<br>
</blockquote>
<br></div>
I expect that it's a lot easier to define a rule where '{' opens a new block on the next line rather than having a 'sometimes it's a single line'-type of exception.</blockquote></div><br>Yes, though</div>
<div class="gmail_extra"><br></div><div class="gmail_extra" style>if (y < 12) {</div><div class="gmail_extra" style>  ierr = Something();CHKERRQ(ierr);</div><div class="gmail_extra" style>}</div><div class="gmail_extra" style>
<br></div><div class="gmail_extra" style>takes three times as many lines. I don't care much either way, but it's nice to not waste vertical space.</div></div>