[issue37141] Allow multiple separators in StreamReader.readuntil

2019-06-03 Thread Andrew Svetlov
Andrew Svetlov added the comment: Yes, Stream supports all StreamReader and StreamWriter methods -- ___ Python tracker ___ ___ Pyth

[issue37141] Allow multiple separators in StreamReader.readuntil

2019-06-03 Thread Bruce Merry
Bruce Merry added the comment: Ok. Does the new Stream still have a similar interface for readuntil i.e. is this still a relevant request against the new API? I'm happy to let deprecated APIs stay as-is. -- ___ Python tracker

[issue37141] Allow multiple separators in StreamReader.readuntil

2019-06-03 Thread Andrew Svetlov
Andrew Svetlov added the comment: Docs will be updated soon. The change has landed a week ago, I had no time for docs update before 3.8 beta. Sorry for that. The idea is: StreamReader and StreamWriter are merged into just Stream, open_connection() is replaced with connect() etc. --

[issue37141] Allow multiple separators in StreamReader.readuntil

2019-06-03 Thread Bruce Merry
Bruce Merry added the comment: I wasn't aware of that deprecation - it doesn't seem to be mentioned at https://docs.python.org/3.8/library/asyncio-stream.html. What is the replacement? -- ___ Python tracker __

[issue37141] Allow multiple separators in StreamReader.readuntil

2019-06-03 Thread Andrew Svetlov
Andrew Svetlov added the comment: Would you make a PR? I guess to modify Stream.readuntil() StreamReader is deprecated, I'm not sure if we should add new functionality to this class. -- ___ Python tracker

[issue37141] Allow multiple separators in StreamReader.readuntil

2019-06-03 Thread Bruce Merry
New submission from Bruce Merry : Text-based protocols sometimes allow a choice of newline separator - I work with one that allows either \r or \n. Unfortunately that doesn't work with StreamReader.readuntil, which only accepts a single separator, so I've had to do some hacky things to obtain