[Python-Dev] Update PEP 7 to require curly braces in C (original) (raw)
Yury Selivanov yselivanov.ml at gmail.com
Tue Jan 19 11:55:43 EST 2016
- Previous message (by thread): [Python-Dev] Update PEP 7 to require curly braces in C
- Next message (by thread): [Python-Dev] Update PEP 7 to require curly braces in C
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 2016-01-18 6:20 PM, Brett Cannon wrote:
On Sun, 17 Jan 2016 at 11:10 Brett Cannon <brett at python.org_ _<mailto:brett at python.org>> 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? Currently this thread stands at: +1 Brett Ethan Robert Georg Nick Maciej Szulik
I guess you forgot to count Barry and me as +1s.
Yury
- Previous message (by thread): [Python-Dev] Update PEP 7 to require curly braces in C
- Next message (by thread): [Python-Dev] Update PEP 7 to require curly braces in C
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]