Looks good in most of the places, but at some places the parenthesis are of course helpful for cohesiveness, and this can be quite subjective. With these patches, are you in general removing all instances of parenthesis when it is not required or also considering for places where parens may help while reading the code?
+1 for the space changes patches. On Sat, Jan 5, 2013 at 8:14 AM, Serhiy Storchaka <rep...@bugs.python.org>wrote: > > Changes by Serhiy Storchaka <storch...@gmail.com>: > > > ---------- > keywords: +patch > Added file: http://bugs.python.org/file28576/minor_things_parens-2.7.diff > Added file: http://bugs.python.org/file28577/minor_things_parens-3.2.diff > Added file: http://bugs.python.org/file28578/minor_things_parens-3.3.diff > Added file: http://bugs.python.org/file28579/minor_things_parens-3.4.diff > Added file: http://bugs.python.org/file28580/minor_things_spaces-3.2.diff > Added file: http://bugs.python.org/file28581/minor_things_spaces-2.7.diff > Added file: http://bugs.python.org/file28582/minor_things_spaces-3.3.diff > Added file: http://bugs.python.org/file28583/minor_things_spaces-3.4.diff > > _______________________________________ > Python tracker <rep...@bugs.python.org> > <http://bugs.python.org/issue16871> > _______________________________________ > _______________________________________________ > Python-bugs-list mailing list > Unsubscribe: > http://mail.python.org/mailman/options/python-bugs-list/senthil%40uthcode.com > >
_______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com