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

[Brett Cannon]

I just checked with Van and we should have CLAs for even PEP contributors,
so it will have to go through the same steps as the other ancillary
repositories.

On Wed, 20 Jan 2016 at 14:41 Brett Cannon <brett at python.org> wrote:

> 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/20160121/70a19d29/attachment.html>