Fix #78883: fgets(STDIN) fails on Windows#4961
Conversation
We add the `is_seekable` member to `php_stdio_stream_data`, and prefer that over `is_pipe`, since the latter is simply a misnomer. We keep `is_pipe` for now for Windows only, though, because we need special support for pipes there. We also fix the misaligned bitfield which formerly took 33 bit.
|
This is an important bugfix for Windows (PHP 7.4.0 can't run any interactive console app), so the PR should be addressed ASAP (IOW, before 7.4.1RC1 is tagged tomorrow). Thanks! |
|
Thanks, @cmb69, for checking on this direction. The old code path would be preserved this way, and One thing yet - if a stream is created not with Thanks. |
|
Thanks for reviewing @weltling!
Sorry, I don't understand. As it's now, |
|
@cmb69, actually I was talking about Thanks. |
|
Thanks! Applied as 996f217. |
|
@weltling any estimate on 7.4.1 release date? |
|
PHP 7.4.1 is scheduled for Dec, 19th. |
We add the
is_seekablemember tophp_stdio_stream_data, and preferthat over
is_pipe, since the latter is simply a misnomer. We keepis_pipefor now for Windows only, though, because we need specialsupport for pipes there. We also fix the misaligned bitfield which
formerly took 33 bit.
This is the cleaner alternative to PR #4952.