<div dir="ltr">On Mon, Jan 21, 2013 at 7:18 PM, Jed Brown <span dir="ltr"><<a href="mailto:jedbrown@mcs.anl.gov" target="_blank">jedbrown@mcs.anl.gov</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="im"><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><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></div>Yes, though</div>
<div class="im">
<div class="gmail_extra"><br></div><div class="gmail_extra">if (y < 12) {</div><div class="gmail_extra">  ierr = Something();CHKERRQ(ierr);</div><div class="gmail_extra">}</div><div class="gmail_extra">
<br></div></div><div class="gmail_extra">takes three times as many lines. I don't care much either way, but it's nice to not waste vertical space.</div></div>
</blockquote></div><br>I agree here.</div><div class="gmail_extra"><br></div><div class="gmail_extra">  Matt<br clear="all"><div><br></div>-- <br>What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>
-- Norbert Wiener
</div></div>