[issue1629] Py_Size() should be named Py_SIZE()

Roundup Robot added the comment: New changeset 63183596db79 by Gregory P. Smith in branch '2.7': Per issue21949 and issue1629: Document the Py_SIZE, Py_TYPE, Py_REFCNT macros. https://hg.python.org/cpython/rev/63183596db79 -- nosy: +python-dev ___ Pyt

[issue1629] Py_Size() should be named Py_SIZE()

Guido van Rossum added the comment: > FWIW, should PyUnicode_Check, PyTuple_Check etc. also change to > PyUnicode_CHECK, PyTuple_CHECK etc because they are macros? While this would technically be the right thing to do, those macros have had a long and productive life before 2.6 so I'd rather be

[issue1629] Py_Size() should be named Py_SIZE()

Martin v. Löwis added the comment: I've fixed PEP 3123 to update the spelling. __ Tracker <[EMAIL PROTECTED]> __ ___ Python-bugs-list mailing list Unsub

[issue1629] Py_Size() should be named Py_SIZE()

Martin v. Löwis added the comment: FWIW, should PyUnicode_Check, PyTuple_Check etc. also change to PyUnicode_CHECK, PyTuple_CHECK etc because they are macros? __ Tracker <[EMAIL PROTECTED]> __

[issue1629] Py_Size() should be named Py_SIZE()

Martin v. Löwis added the comment: Why the compatibility? __ Tracker <[EMAIL PROTECTED]> __ ___ Python-bugs-list mailing list Unsubscribe: http://mail.

[issue1629] Py_Size() should be named Py_SIZE()

Christian Heimes added the comment: I've renamed the three macros and added the old names for b/w compatibility. -- resolution: accepted -> fixed status: open -> closed __ Tracker <[EMAIL PROTECTED]>

[issue1629] Py_Size() should be named Py_SIZE()

Changes by Guido van Rossum: -- resolution: -> accepted __ Tracker <[EMAIL PROTECTED]> __ ___ Python-bugs-list mailing list Unsubscribe: http

[issue1629] Py_Size() should be named Py_SIZE()

Christian Heimes added the comment: I tried a simple find | xargs sed replace and it worked well. find -name '*.c' -or -name '*.h' -or -name '*.rst' | xargs sed -i s/Py_Size\(/Py_SIZE\(/g __ Tracker <[EMAIL PROTECTED]> _

[issue1629] Py_Size() should be named Py_SIZE()

Christian Heimes added the comment: I think it would be easier to merge from trunk to py3k before the change and skip the revision in the next merge. The rename can be done with a simple find | xargs sed -i. A merge might be more painful. -- nosy: +tiran

[issue1629] Py_Size() should be named Py_SIZE()

Martin v. Löwis added the comment: I assume that applies to Py_Refcnt and Py_Type as well? __ Tracker <[EMAIL PROTECTED]> __ ___ Python-bugs-list mailing

[issue1629] Py_Size() should be named Py_SIZE()

Guido van Rossum added the comment: Agreed. -- nosy: +gvanrossum __ Tracker <[EMAIL PROTECTED]> __ ___ Python-bugs-list mailing list Unsubscrib

[issue1629] Py_Size() should be named Py_SIZE()

New submission from Raymond Hettinger: It would be helpful to have the name provide a cue that a macro is being used. -- assignee: loewis components: Extension Modules messages: 58634 nosy: loewis, rhettinger severity: normal status: open title: Py_Size() should be named Py_SIZE() versio