Remove hacks, pass script directly to stdin

Author: cursedcoder

EProcess/Adapter/BaseAdapter.php

@@ -2,7 +2,6 @@
 
 namespace EProcess\Adapter;
 
-use EProcess\Terminator;
 use React\EventLoop\LoopInterface;
 
 abstract class BaseAdapter

EProcess/Adapter/ChildProcess.php

@@ -59,21 +59,22 @@ public function create($class, array $data = [])
         $unix = $this->createUnixSocket();
         $messenger = MessengerFactory::server($unix, $this->loop);
 
-        $file = sprintf(__DIR__ . '/../../tmp/%s.php', $this->node);
-
-        file_put_contents($file, sprintf(
+        $script = sprintf(
             $this->script,
             EPROCESS_AUTOLOAD,
             $unix,
             $class,
             base64_encode($this->serialize($data))
-        ));
+        );
 
-        $this->process = new Process(sprintf('exec %s %s', $php, realpath($file)));
-        $this->process->start($this->loop);
+        $this->process = new Process($php);
+        $this->process->start($this->loop, 0.1);
+        
+        $this->process->stdin->resume();
+        $this->process->stdin->write($script);
 
-        $this->loop->addTimer(1, function() use ($file) {
-            unlink($file);
+        $this->process->stdin->on('full-drain', function() {
+            $this->process->stdin->close();
         });
 
         $this->process->stdout->on('data', function($data) {

EProcess/Adapter/SymfonyProcess.php

@@ -44,7 +44,6 @@ class SymfonyProcess extends BaseAdapter
 }
 PHP;
 
-    private $loop;
     private $process;
 
     public function create($class, array $data = [])

EProcess/Stream/FullDrainSteam.php

@@ -0,0 +1,51 @@
+<?php
+
+namespace EProcess\Stream;
+
+use React\Stream\Stream;
+use React\EventLoop\LoopInterface;
+use React\Stream\Buffer;
+
+class FullDrainStream extends Stream
+{
+
+    public function __construct($stream, LoopInterface $loop)
+    {
+        $this->stream = $stream;
+        if (!is_resource($this->stream) || get_resource_type($this->stream) !== "stream") {
+            throw new \InvalidArgumentException('First parameter must be a valid stream resource');
+        }
+
+        stream_set_blocking($this->stream, 0);
+
+        // Use unbuffered read operations on the underlying stream resource.
+        // Reading chunks from the stream may otherwise leave unread bytes in
+        // PHP's stream buffers which some event loop implementations do not
+        // trigger events on (edge triggered).
+        // This does not affect the default event loop implementation (level
+        // triggered), so we can ignore platforms not supporting this (HHVM).
+        if (function_exists('stream_set_read_buffer')) {
+            stream_set_read_buffer($this->stream, 0);
+        }
+
+        $this->loop = $loop;
+        $this->buffer = new Buffer($this->stream, $this->loop);
+
+        $that = $this;
+
+        $this->buffer->on('error', function ($error) use ($that) {
+            $that->emit('error', array($error, $that));
+            $that->close();
+        });
+
+        $this->buffer->on('drain', function () use ($that) {
+            $that->emit('drain', array($that));
+        });
+
+        $this->buffer->on('full-drain', function () use ($that) {
+            $that->emit('full-drain', array($that));
+        });
+
+        $this->resume();
+    }
+}

README.md

@@ -71,9 +71,3 @@ ApplicationFactory::launch(Main::class);
 ```
 
 You need to have proper autoloading established in order to use this example.
-
-ToDo
-=======
-
-* Refactor adapters, remove hacks, pass php code directly with stdin instead of file.
-* Can IPC be done without unix sockets or using only single socket?