[Python-Dev] Update PEP 7 to require curly braces in C

[Brett Cannon]

On Wed, 20 Jan 2016 at 14:31 Guido van Rossum <guido at python.org> wrote:

> The wording is totally fine! You might still want to revert it and
> re-commit it so it doesn't look like a mistake when reviewing the log.
>

Sure thing!


>
> BTW When can we start using git for the peps repo?
>

Depends on how fast the parts covered in
https://www.python.org/dev/peps/pep-0512/#requirements-for-code-only-repositories
 and
https://www.python.org/dev/peps/pep-0512/#requirements-for-web-related-repositories
takes.
My hope is before PyCon US (although if we choose to not enforce the CLA on
PEP contributions then it could happen even faster).

-Brett


>
> On Wed, Jan 20, 2016 at 12:18 PM, Brett Cannon <brett at python.org> wrote:
>
>>
>>
>> On Wed, 20 Jan 2016 at 10:45 Terry Reedy <tjreedy at udel.edu> wrote:
>>
>>> On 1/20/2016 12:45 PM, Brett Cannon wrote:
>>> >
>>> >
>>> > On Tue, 19 Jan 2016 at 19:33 Martin Panter <vadmium+py at gmail.com
>>> > <mailto:vadmium%2Bpy at gmail.com>> wrote:
>>> >
>>> >     On 19 January 2016 at 20:12, Brett Cannon <brett at python.org
>>> >     <mailto: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).
>>>
>>> I thought that the above was your intentional compromise change given
>>> the range of opinions ;-).
>>>
>>
>> It is, but Guido is the author of PEP 7 and so I didn't want to check in
>> the change without his approval first.
>>
>> _______________________________________________
>> Python-Dev mailing list
>> Python-Dev at python.org
>> https://mail.python.org/mailman/listinfo/python-dev
>>
> Unsubscribe:
>> https://mail.python.org/mailman/options/python-dev/guido%40python.org
>>
>>
>
>
> --
> --Guido van Rossum (python.org/~guido)
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.python.org/pipermail/python-dev/attachments/20160120/b13fd417/attachment.html>