[Python-Dev] PEP 7 and braces { .... } on if

MRAB python at mrabarnett.plus.com
Mon Jun 5 12:54:23 EDT 2017


On 2017-06-05 13:00, Skip Montanaro wrote:
> On Mon, Jun 5, 2017 at 12:41 AM, Serhiy Storchaka <storchaka at gmail.com> wrote:
>> Barry and Victor prefer moving a brace on a new line in all multiline
>> conditional cases. I think that it should be done only when the condition
>> continuation lines and the following block of the code have the same
>> indentation (as in the example above), and the following code is enough
>> readable:
>>
>>         if (PyErr_WarnFormat(PyExc_DeprecationWarning, 1,
>>                              "invalid escape sequence '\\%c'",
>>                              *first_invalid_escape) < 0) {
>>             Py_DECREF(result);
>>             return NULL;
>>         }
>>
>> What other core developers think about this?
> 
> Wow, this discussion takes me back. Glad I don't have to check out
> comp.lang.c to get my brace placement fix. <wink>
> 
FWIW, I half-indent continuation lines (an advantage of using spaces 
instead of tabs).


More information about the Python-Dev mailing list