Merge pull request #156 from WyriHaximus-labs/ext-event_FEATURE_FDS_enabled_by_default

ExtEventLoop: Remove optional EventBaseConfig and make FEATURE_FDS enabled by default

Author: jsor

src/ExtEventLoop.php

@@ -36,8 +36,11 @@ final class ExtEventLoop implements LoopInterface
     private $signals;
     private $signalEvents = array();
 
-    public function __construct(EventBaseConfig $config = null)
+    public function __construct()
     {
+        $config = new EventBaseConfig();
+        $config->requireFeatures(EventBaseConfig::FEATURE_FDS);
+
         $this->eventBase = new EventBase($config);
         $this->futureTickQueue = new FutureTickQueue();
         $this->timerEvents = new SplObjectStorage();

tests/ExtEventLoopTest.php

@@ -16,13 +16,7 @@ public function createLoop($readStreamCompatible = false)
             $this->markTestSkipped('ext-event tests skipped because ext-event is not installed.');
         }
 
-        $cfg = null;
-        if ($readStreamCompatible) {
-            $cfg = new \EventConfig();
-            $cfg->requireFeatures(\EventConfig::FEATURE_FDS);
-        }
-
-        return new ExtEventLoop($cfg);
+        return new ExtEventLoop();
     }
 
     public function createStream()