You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee='https://github.com/serhiy-storchaka'closed_at=<Date2016-07-07.15:21:19.226>created_at=<Date2016-06-16.16:33:33.045>labels= ['type-bug', 'expert-argument-clinic']
title='Clinic: first parameter for module-level functions should be PyObject*, not PyModuleDef*'updated_at=<Date2016-07-07.15:21:19.225>user='https://github.com/encukou'
Currently, Argument Clinic generates "PyModuleDef * module" for the first argument of module-level functions. But, the functions are passed the actual module object, not the ModuleDef.
The correct type to use is PyObject*, which is used for modules in the PyModule_* API.
(It turns out nothing in core Python currently uses the argument except passing it to other _impl functions, but this could change as PEP-489 is improved upon.)
The attached patch contains manual changes only. Please run make clinic after applying it to regenerate allll the code.
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: