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

[Sean Farley]

On Mon, Jan 21, 2013 at 9:26 PM, Satish Balay <balay at mcs.anl.gov> wrote:
> 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.

So, am I the only one that like the uniformity of the three-liner?