Eric Snow added the comment:

What's the benefit to adding PyInterpreterState_GetModuleCache()?  TBH, it 
should only be needed in this short period during startup when the import 
system hasn't been bootstrapped yet.  After that code can import sys and access 
sys.modules from there.  (For that matter, PyImport_GetModuleDict() doesn't buy 
all that much either...)  I think all this would be clearer in a world with PEP 
432.  :)

FWIW, I'm inclined to encourage new APIs where it makes sense that take an 
explicit interp arg.  I just don't think a new public API is warranted here.  
If we didn't already have PyImport_GetModuleDict(), I'd probably just move the 
code to Python/pylifecycle.c, inlined or in a small static function.

And +1 to ModuleCache vs. ModuleDict. :)

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue28411>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to