Message 362602 - Python tracker

➜

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	BTaskaya
Recipients	BTaskaya, larry, pablogsal
Date	2020-02-24.20:07:45
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<1582574865.92.0.664787164093.issue39741@roundup.psfhosted.org>
In-reply-to

Content
After preparing the patch and transforming all arguments with a __clinic_ prefix, I saw there are some actions that are taken by relying on the parser code. An example; https://github.com/python/cpython/blob/8af4712a16e4b7d1b60f1faec13cd7a88da95f6a/Objects/codeobject.c#L616-L631 In that case, self was already replaced with __clinic_self so that code doesn't work. IMHO there should be an identifier to distinguish these cases like; <self>->co_argc etc. When we see such defaults we can just replace them with the prefixed version.

After preparing the patch and transforming all arguments with a __clinic_ prefix, I saw there are some actions that are taken by relying on the parser code. An example;
https://github.com/python/cpython/blob/8af4712a16e4b7d1b60f1faec13cd7a88da95f6a/Objects/codeobject.c#L616-L631

In that case, self was already replaced with __clinic_self so that code doesn't work. IMHO there should be an identifier to distinguish these cases like; <self>->co_argc etc. When we see such defaults we can just replace them with the prefixed version.

History
Date	User	Action	Args
2020-02-24 20:07:45	BTaskaya	set	recipients: + BTaskaya, larry, pablogsal
2020-02-24 20:07:45	BTaskaya	set	messageid: <1582574865.92.0.664787164093.issue39741@roundup.psfhosted.org>
2020-02-24 20:07:45	BTaskaya	link	issue39741 messages
2020-02-24 20:07:45	BTaskaya	create