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.23:24:36
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1481066676.61.0.877764894874.issue28838@psf.upfronthosting.co.za>
In-reply-to
Content
Serhiy:
> Seems most of comments that did not follow common style are your comments to  "fast call" functions. It would be better to fix only that comments.

I would like to modify abstract.h because its weird style annoy me for years. I would like to make the header consistent with all other headers.

I checked unicodeobject.h, code.h, frameobject.h, floatobject.h, etc.: comments are always before declaration.

About "fast calls" in abstract.h, sorry, it's my fault. When I added new code, I used the same style than all other files. I didn't notice that abstract.h has a special style. That's why I would like to make it consistent again.


Serhiy:
> Reviewed patch did not contain committed changes.

You wrote on the review that a comment is misplaced: "The comment is related to PyObject_Call() declared 85 lines above."

That's why I took the opportunity of a refactoring change to fix comments.


Serhiy:
> That changes are not tiny.

Between rename.patch and the pushed change e6ad41a3f3bd, I only moved 5 comments: see attached rename_pushed.patch which is the diff between the two patches.

For me, it was obvious that the comment must be before the declaration, since it is the style used in all other Python headers. Sorry, I didn't expect that you didn't want to change that. I misunderstood your comment on the review.
History
Date User Action Args
2016-12-06 23:24:36vstinnersetrecipients: + vstinner, ncoghlan, eric.smith, r.david.murray, python-dev, serhiy.storchaka
2016-12-06 23:24:36vstinnersetmessageid: <1481066676.61.0.877764894874.issue28838@psf.upfronthosting.co.za>
2016-12-06 23:24:36vstinnerlinkissue28838 messages
2016-12-06 23:24:36vstinnercreate