> Another candidate that could probably go away, regardless of the
> import work, is 'modules_by_index'. As far as I can see, there is
> only one use of interp->modules_by_index in the cpython code-base:
> PyState_FindModule() in Python/pystate.c. Likewise there is only one
> use of PyState_FindModule(): atexit_callfuncs() in
> Modules/atexitmodule.c.
There will be certainly many more uses of PySteate_FindModule in the
future. I fail to see what is gained by this kind of change, and am
firmly -1 on removing variables arbitrarily. |