Guido van Rossum added the comment:

I just re-read the docs here (which came straight from the open_connection() 
docstring). The parenthetical remark is confusing -- I meant something more like

"""
If you want to use different classes than StreamReader, StreamReaderProtocol 
and StreamWriter, just copy the code of open_connection() and substitute your 
own classes.
"""

That is, the customization it is trying to refer to is open_connection(), not 
the classes.

FWIW I think subclassing implementation classes is a dangerous practice and 
shouldn't be encouraged.

Because the same reasoning applies to all the top-level functions here (but not 
to the classes), I think I'll delete the offending paragraph and replace it 
with something like the following at the top of the docs for this module:

"""
The top-level functions in this module are meant convenience wrappers only; 
there's really nothing special there, and if they don't do what you want feel 
free to copy their code.
"""

----------

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

Reply via email to