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 alexandre.vassalotti
Recipients Arfrever, alexandre.vassalotti, asvetlov, mstefanro, neologix, pitrou, rhettinger, serhiy.storchaka
Date 2013-05-12.08:53:44
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <>
Stefan, I took a quick look at your patch. There is a couple things that stands out.

First, I think the implementation of BINGLOBAL and BINGLOBAL_BIG should be moved to another patch. Adding a binary version opcode for GLOBAL is a separate feature and it should be reviewed independently. Personally, I prefer the STACK_GLOBAL opcode I proposed as it much simpler to implement, but I am biased.

Next, the patch's formatting should be fixed to conform to PEP 7 and PEP 8. Make sure the formatting is consistent with the surrounding code. In particular, comments should be full sentences that explains why we need this code. Avoid adding comments that merely say what the code does, unless the code is complex.

In addition, please replace the uses of PyUnicode_InternFromString with the _Py_IDENTIFIER as needed. The latter allow the static strings to be garbage collected when the module is deleted, which is friendlier to embedded interpreters. It is also lead to cleaner code.

Finally, the class method check hack looks like a bug to me. There are multiple solutions here. For example, we could fix class methods to be cached so they always have the same ID once they are created. Or, we could remove the 'is' check completely if it is unnecessary.
Date User Action Args
2013-05-12 08:53:45alexandre.vassalottisetrecipients: + alexandre.vassalotti, rhettinger, pitrou, Arfrever, asvetlov, neologix, serhiy.storchaka, mstefanro
2013-05-12 08:53:45alexandre.vassalottisetmessageid: <>
2013-05-12 08:53:45alexandre.vassalottilinkissue17810 messages
2013-05-12 08:53:44alexandre.vassalotticreate