Łukasz Langa <luk...@langa.pl> added the comment:

> if you ask me, I would change all of them

The problem with adopting an ABC base class for each such use case is two-fold:
- it changes the MRO putting the ABCMeta metaclass in which might make existing 
subclasses suddenly incompatible; and
- is costly in terms of performance even if subclasses are still compatible.

In other words, putting ABC in isn't a transparent operation. This is one 
reason why it wasn't included in asyncio code.

Since we're not reacting to any breakage here, I'd feel best leaving this 
alone. Thanks for your care Anton!

----------
resolution:  -> rejected
stage: patch review -> resolved
status: open -> closed

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

Reply via email to