STINNER Victor added the comment:

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.

----------
Added file: http://bugs.python.org/file45779/rename_pushed.patch

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue28838>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to