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

[Satish Balay]

On Mon, 21 Jan 2013, Matthew Knepley wrote:

> On Mon, Jan 21, 2013 at 7:18 PM, Jed Brown <jedbrown at mcs.anl.gov> wrote:
> 
> >
> > 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.
> 
> I agree here.

I've prefered the 1 liners as well.

Satish