| $this->stdin = new WritableResourceStream($this->pipes[0], $loop); | ||
| $this->stdin->on('close', function () { | ||
| if (is_resource($this->pipes[0])) { | ||
| fclose($this->pipes[0]); |
| if (is_resource($that->pipes[0])) { | ||
| fclose($that->pipes[0]); | ||
| } | ||
| }); |
Update react/stream to support 0.6, 0.7, and 1.x
Update react/stream to support 0.7.2 and future 1.x
|
|
||
| Once a process is started, its I/O streams will be constructed as instances of | ||
| `React\Stream\Stream`. Before `start()` is called, these properties are `null`. | ||
| `React\Stream\ReadableResourceStream` as `React\Stream\WritableResourceStream`. Before `start()` is called, these properties are `null`. |
| Each of these implement the underlying | ||
| [`DuplexStreamInterface`](https://github.com/reactphp/stream#duplexstreaminterface) | ||
| and you can use any of its events and methods as usual: | ||
| [`ReadableResourceStream`](https://github.com/reactphp/stream#readablestreaminterface) or [`WritableResourceStream`](https://github.com/reactphp/stream#writablestreaminterface) |
|
|
||
| For more details, see the | ||
| [`DuplexStreamInterface`](https://github.com/reactphp/stream#duplexstreaminterface). | ||
| [`ReadableResourceStream`](https://github.com/reactphp/stream#readablestreaminterface) and [`WritableResourceStream`](https://github.com/reactphp/stream#writablestreaminterface). |
WyriHaximus added a commit to WyriHaximus-labs/child-process that referenced this pull request
Jun 17, 2017
WyriHaximus added a commit to WyriHaximus-labs/child-process that referenced this pull request
Jun 17, 2017
PHP 5.3 fix Updated to react/stream:^0.7.2 Removed unused React\Stream\Stream import Updated readme to the updated streams As => and reactphp#38 (comment) Use interfaces instead of concrete classes in the documentation reactphp#38 (comment) Line folding
clue approved these changes Jun 17, 2017
jsor approved these changes Jun 18, 2017
Merged
Closed