Eric Snow <ericsnowcurren...@gmail.com> added the comment:

Perhaps PyImport_GetModule() should aquire-release the module's lock before the 
lookup?  This would effectively be a call to _lock_unlock_module() in 
importlib._bootstrap.

The alternative is to encourage using PyImport_Import() instead, like the PR 
has done.  In the case the docs for PyImport_GetModule() should make it clear 
that it is guaranteed that the module is fully imported yet (and recommend 
using PyImport_Import() for the guarantee).

Either way there should be a new issue for the more general change (and it 
should reference this issue).

----------
type: crash -> behavior

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

Reply via email to