Message341767
| Author |
josh.r |
| Recipients |
Leo Singer, Marcel H2, bethard, eli.bendersky, evan_, josh.r, markgrandi, moritz, naufraghi, palaviv, paul.j3, r.david.murray, sedrubal, serhiy.storchaka |
| Date |
2019年05月07日.16:18:25 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1557245905.78.0.226705545515.issue14156@roundup.psfhosted.org> |
| In-reply-to |
| Content |
Serhiy: To be clear, Moritz's patch won't work if stdin/stdout are io.StringIO or the like either, since StringIO lacks a buffer attribute.
Personally, I'm content to:
1. Ignore the closeability of the standard handles; that's not part of this bug, and just introduces more headaches to getting a patch approved (and conceivably breaks back compat if the user *wants* the handle closed even if it's a standard handle). Nothing is made worse by ignoring it after all, just not made better immediately.
2. Ignore the possibility of stdin/stdout without a buffer attribute; it will raise an error, but that's a reasonable response to demanding a binary stream in a scenario where only a text stream is available
3. Not use fileno, as it mostly supports the "anti-close" behavior I dismissed in #1, and has a bunch of pitfalls (e.g. a standard handle rebound to a TextIOWrapper around GzipFile or the like has a fileno attribute, but using it bypasses the compression; basically, you can't consider fileno to be equivalent to the underlying binary stream, because binary streams can be wrapped as well).
I'm going to convert Moritz's proposal to a PR and hope I can get core developer approval for it. |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2019年05月07日 16:18:25 | josh.r | set | recipients:
+ josh.r, bethard, naufraghi, r.david.murray, eli.bendersky, paul.j3, serhiy.storchaka, moritz, markgrandi, palaviv, evan_, sedrubal, Marcel H2, Leo Singer |
| 2019年05月07日 16:18:25 | josh.r | set | messageid: <1557245905.78.0.226705545515.issue14156@roundup.psfhosted.org> |
| 2019年05月07日 16:18:25 | josh.r | link | issue14156 messages |
| 2019年05月07日 16:18:25 | josh.r | create |
|