[Python-Dev] Update PEP 7 to require curly braces in C
Yury Selivanov
yselivanov.ml at gmail.com
Mon Jan 18 11:52:08 EST 2016
I'm usually fine with code like this:
if ( ... )
return
But when there is a multi-line 'else' clause, I just want to use braces
for both 'if' and its 'else'. So, for consistency, I'm +1 for
recommending to use braces everywhere.
And +1 for requiring to use braces if one of the clauses of the 'if'
statement is multi-line.
Yury
On 2016-01-17 2:10 PM, Brett Cannon wrote:
> While doing a review of http://bugs.python.org/review/26129/ I asked
> to have curly braces put around all `if` statement bodies. Serhiy
> pointed out that PEP 7 says curly braces are optional:
> https://www.python.org/dev/peps/pep-0007/#id5. I would like to change
> that.
>
> My argument is to require them to prevent bugs like the one Apple made
> with OpenSSL about two years ago:
> https://www.imperialviolet.org/2014/02/22/applebug.html. Skipping the
> curly braces is purely an aesthetic thing while leaving them out can
> lead to actual bugs.
>
> Anyone object if I update PEP 7 to remove the optionality of curly
> braces in PEP 7?
>
>
> _______________________________________________
> 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/yselivanov.ml%40gmail.com
More information about the Python-Dev
mailing list