[Python-Dev] Update PEP 7 to require curly braces in C
Brett Cannon
brett at python.org
Wed Jan 20 12:45:11 EST 2016
On Tue, 19 Jan 2016 at 19:33 Martin Panter <vadmium+py at gmail.com> wrote:
> On 19 January 2016 at 20:12, Brett Cannon <brett at python.org> wrote:
> > Here is a proposed update:
> >
> > diff -r 633f51d10a67 pep-0007.txt
> > --- a/pep-0007.txt Mon Jan 18 10:52:57 2016 -0800
> > +++ b/pep-0007.txt Tue Jan 19 12:11:44 2016 -0800
> > @@ -75,9 +75,9 @@
> > }
> >
> > * Code structure: one space between keywords like ``if``, ``for`` and
> > - the following left paren; no spaces inside the paren; braces may be
> > - omitted where C permits but when present, they should be formatted
> > - as shown::
> > + the following left paren; no spaces inside the paren; braces are
> > + strongly preferred but may be omitted where C permits, and they
> > + should be formatted as shown::
> >
> > if (mro != NULL) {
> > ...
>
> This change seems to be accidentally smuggled in, in the guise of a
> PEP 512 update :)
>
Darn, sorry about that; forgot I had that change sitting in my peps
checkout. I'll revert it when I get home (unless the change is actually
acceptable to Guido).
-Brett
>
> My view is I prefer always using curly brackets in my own code. It is
> easier to add printf() debugging without making mistakes. I support
> “strongly preferring” them in the style guide, which is as much as a
> style guide can do anyway.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.python.org/pipermail/python-dev/attachments/20160120/1139362c/attachment.html>
More information about the Python-Dev
mailing list