Barry A. Warsaw <ba...@python.org> added the comment:

Hi Serhiy.  I'm curious whether this is a pure clean up or if there are actual 
problems you're trying to fix.

* I'm okay with using _PyObject_GetBuiltin() but it does bother me in general 
to use too many non-public (and thus undocumented) APIs.  It makes 
understanding what's going on more difficult.  This is a general gripe of mine 
with Python's C API.

* Given that removing support for callable modules is technically a backward 
incompatible change, is it worth it?  If we had done it before 3.7 was 
released, it would have been fine, but now we can't guarantee it won't break 
someone's code.  I agree it's unlikely, but it is possible.

* Perhaps we should explicitly pass through KeyboardInterrupt and MemoryError 
rather than just ignoring ImportError and AttributeError.

----------

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

Reply via email to