[petsc-dev] Style Guide: How to format single-line if/for/while-blocks?

Jed Brown jedbrown at mcs.anl.gov
Mon Jan 21 19:18:52 CST 2013


On Mon, Jan 21, 2013 at 7:05 PM, Karl Rupp <rupp at mcs.anl.gov> wrote:

>      Things like
>>
>>      if (y < 12) {ierr = Something(); CHKERRQ(ierr);}
>>
>> don't match the standard but …. am I being too picky? I believe the PETSc
>> make uncrustify rule would move that to separate lines.
>>
>
> 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.


Yes, though

if (y < 12) {
  ierr = Something();CHKERRQ(ierr);
}

takes three times as many lines. I don't care much either way, but it's
nice to not waste vertical space.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mcs.anl.gov/pipermail/petsc-dev/attachments/20130121/5c4d1cb5/attachment.html>


More information about the petsc-dev mailing list