This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author vstinner
Recipients eric.smith, ncoghlan, python-dev, r.david.murray, serhiy.storchaka, vstinner
Date 2016-12-06.17:45:18
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1481046321.85.0.862787234017.issue28838@psf.upfronthosting.co.za>
In-reply-to
Content
I pushed another large patch to "fix" the indentation of abstract.h. I would prefer to also have this "reindent" change on review, but our reviewing tool doesn't support patch series :-(


Attached cleanup.patch reformats deeply the header file to use the same style for all comments, use the same style used in other Python headers. IMO it makes the header file more consistent with other header files and it makes the file more readable.


R. David Murray (r.david.murray):
> Uniformize" isn't really an English word :)

Oh, it's a frenglish word in this case :-) Good to know.


Serhiy Storchaka: "You have moved some comments and now some descriptions are placed below the corresponding declaration, and some descriptions are placed above. Now it is not clear to what function every comment is related."

It don't think that my change makes abstract.h worse, some comments were far from their declaration, and there is was no unique style for comments. It's a mess.


Serhiy: "You should either revert your changes or change *all* headers by moving *all* descriptions above corresponding declarations."

My plan is to rewrite completely abstact.h, this file annoys me since many months :-)


"That would be very large patch, and I doubt it will be approved."

Why not?


"Please don't make such large changes without a review."

You reviewed rename.patch, I only made tiny changes after this patch. As I explained, it's hard to work on such large patch.
History
Date User Action Args
2016-12-06 17:45:22vstinnersetrecipients: + vstinner, ncoghlan, eric.smith, r.david.murray, python-dev, serhiy.storchaka
2016-12-06 17:45:21vstinnersetmessageid: <1481046321.85.0.862787234017.issue28838@psf.upfronthosting.co.za>
2016-12-06 17:45:21vstinnerlinkissue28838 messages
2016-12-06 17:45:18vstinnercreate