Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/usr.bin/xlint



On 27/03/2021 21:59, Christos Zoulas wrote:
In article <12312.1616877523%splode.eterna.com.au@localhost>,
matthew green  <mrg%eterna.com.au@localhost> wrote:
Sounds good.  Will it mean that the following code is also preferred to
use parentheses?  I like the simplicity without the parentheses, but for
consistency and simplicity of the rule set, I'd also accept the
parenthesized version.

      snprintf(buf, sizeof buf, "%s%s", arg1, arg2);

FWIW, there's very little chance i'm *not* going to use this form.

i've always applied the general () rule here -- don't have them
unless it's either necessary *or* useful for comprehension, and
that rule clearly says do not use them for the above case.  so
maybe "prefer paren unless clear", allowing objective takes ;)

I would go for consistency; making grep and sed easier trumps aesthetic >
christos

What Christos said.

Nick




Home | Main Index | Thread Index | Old Index