ackintosh / snidel

src/Snidel/ForkCollection.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     public function toArray()
     {
         return array_map(
-            function ($fork) {
+            function($fork) {
                 return $fork->getResult()->getReturn();
             },
             $this->forks

src/Snidel/Log.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
                 $type,
                 $pid,
                 ($this->ownerPid === $pid) ? 'p' : 'c',
-                $message . PHP_EOL
+                $message.PHP_EOL
             )
         );
     }

Switch Indentation +9 added lines, -9 removed lines

@@ -50,15 +50,15 @@
         }
         $pid = getmypid();
         switch (true) {
-        case $this->ownerPid === $pid:
-            $role = 'owner';
-            break;
-        case $this->masterPid !== null && $this->masterPid === $pid:
-            $role = 'master';
-            break;
-        default:
-            $role = 'worker';
-            break;
+            case $this->ownerPid === $pid:
+                $role = 'owner';
+                break;
+            case $this->masterPid !== null && $this->masterPid === $pid:
+                $role = 'master';
+                break;
+            default:
+                $role = 'worker';
+                break;
         }
         fputs(
             $this->destination,

src/Snidel/Token.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
     {
         $this->ownerPid = $ownerPid;
         $this->concurrency = $concurrency;
-        $this->ipcKey = new IpcKey($ownerPid, 'snidel_token_' . uniqid((string) mt_rand(1, 100), true));
+        $this->ipcKey = new IpcKey($ownerPid, 'snidel_token_'.uniqid((string) mt_rand(1, 100), true));
         $this->id = msg_get_queue($this->ipcKey->generate());
         $this->initializeQueue();
     }

src/Snidel/MapContainer.php

Spacing +1 added lines, -1 removed lines

@@ -137,6 +137,6 @@
             return $index;
         }
 
-        throw new MapContainerException('childPid not found. pid: ' . $childPid);
+        throw new MapContainerException('childPid not found. pid: '.$childPid);
     }
 }

src/Snidel/ResultQueue.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace Ackintosh\Snidel;
 
 use Ackintosh\Snidel\IpcKey;
-use Opis\Closure\SerializableClosure;
 
 class ResultQueue
 {

Doc Comments +4 added lines

@@ -12,6 +12,9 @@ 
 
     private $dequeuedCount = 0;
 
+    /**
+     * @param integer $ownerPid
+     */
     public function __construct($ownerPid)
     {
         $this->ownerPid = $ownerPid;
@@ -21,6 +24,7 @@ 
 
     /**
      * @param   \Ackintosh\Snidel\Fork
+     * @param Fork $fork
      */
     public function enqueue($fork)
     {

src/Snidel/TaskQueue.php

Doc Comments +3 added lines

@@ -11,6 +11,9 @@
     private $queuedCount = 0;
     private $dequeuedCount = 0;
 
+    /**
+     * @param integer $ownerPid
+     */
     public function __construct($ownerPid)
     {
         $this->ownerPid = $ownerPid;

src/Snidel/Task.php

Doc Comments +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
     /**
      * @param   callable    $callable
      * @param   array       $args
-     * @param   string      $string
+     * @param string|null $tag
      */
     public function __construct($callable, $args, $tag)
     {
@@ -60,7 +60,7 @@ 
     }
 
     /**
-     * @return  string|null
+     * @return  string
      */
     public function getTag()
     {

src/Snidel/IpcKey.php

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
      */
     public function generate()
     {
-        $pathname = '/tmp/' . $this->prefix . $this->ownerPid;
+        $pathname = '/tmp/'.$this->prefix.$this->ownerPid;
         if (!file_exists($pathname)) {
             touch($pathname);
         }
@@ -52,8 +52,8 @@ 
      */
     public function delete()
     {
-        if (file_exists('/tmp/' . $this->prefix . $this->ownerPid)) {
-            unlink('/tmp/' . $this->prefix . $this->ownerPid);
+        if (file_exists('/tmp/'.$this->prefix.$this->ownerPid)) {
+            unlink('/tmp/'.$this->prefix.$this->ownerPid);
         }
     }
 }

src/Snidel/ForkContainer.php

Doc Comments +9 added lines, -3 removed lines

@@ -45,6 +45,7 @@ 
 
     /**
      * @param   int     $ownerPid
+     * @param Log $log
      */
     public function __construct($ownerPid, $log, $concurrency = 5)
     {
@@ -60,6 +61,7 @@ 
 
     /**
      * @param   \Ackintosh\Snidel\Task
+     * @param Task $task
      * @return  void
      */
     public function enqueue($task)
@@ -155,6 +157,7 @@ 
      * fork worker process
      *
      * @param   \Ackintosh\Snidel\Task
+     * @param Task $task
      * @return  void
      * @throws  \RuntimeException
      */
@@ -262,7 +265,7 @@ 
     }
 
     /**
-     * @return  array
+     * @return  integer[]
      */
     public function getChildPids()
     {
@@ -280,6 +283,9 @@ 
         return $this->forks[$pid];
     }
 
+    /**
+     * @param string $tag
+     */
     public function getCollection($tag = null)
     {
         if ($tag === null) {
@@ -293,7 +299,7 @@ 
      * return forks
      *
      * @param   string  $tag
-     * @return  \Ackintosh\Snidel\Fork[]
+     * @return  ForkCollection
      */
     private function getCollectionWithTag($tag)
     {
@@ -313,7 +319,7 @@ 
     }
 
     /**
-     * @return  \Ackintosh\Sniden\Error
+     * @return  Error
      */
     public function getError()
     {

Spacing +8 added lines, -8 removed lines

@@ -129,7 +129,7 @@ 
 
         if ($pid) {
             // owner
-            $this->log->info('pid: ' . getmypid());
+            $this->log->info('pid: '.getmypid());
 
             return $this->masterPid;
         } elseif ($pid === -1) {
@@ -137,14 +137,14 @@ 
         } else {
             // master
             $taskQueue = new TaskQueue($this->ownerPid);
-            $this->log->info('pid: ' . $this->masterPid);
+            $this->log->info('pid: '.$this->masterPid);
 
             foreach ($this->signals as $sig) {
                 $this->pcntl->signal($sig, SIG_DFL, true);
             }
 
             while ($task = $taskQueue->dequeue()) {
-                $this->log->info('dequeued task #' . $taskQueue->dequeuedCount());
+                $this->log->info('dequeued task #'.$taskQueue->dequeuedCount());
                 if ($this->token->accept()) {
                     $this->forkWorker($task);
                 }
@@ -171,10 +171,10 @@ 
 
         if (getmypid() === $this->masterPid) {
             // master
-            $this->log->info('forked worker. pid: ' . $fork->getPid());
+            $this->log->info('forked worker. pid: '.$fork->getPid());
         } else {
             // worker
-            $this->log->info('has forked. pid: ' . getmypid());
+            $this->log->info('has forked. pid: '.getmypid());
             // @codeCoverageIgnoreStart
 
             foreach ($this->signals as $sig) {
@@ -182,7 +182,7 @@ 
             }
 
             $resultQueue = new ResultQueue($this->ownerPid);
-            register_shutdown_function(function () use ($fork, $resultQueue) {
+            register_shutdown_function(function() use ($fork, $resultQueue) {
                 if ($fork->hasNoResult() || !$fork->isQueued()) {
                     $result = new Result();
                     $result->setFailure();
@@ -315,8 +315,8 @@ 
      */
     private function getCollectionWithTag($tag)
     {
-        $collection = array_filter($this->forks, function ($fork) use ($tag) {
-            return $fork->getTag() ===  $tag;
+        $collection = array_filter($this->forks, function($fork) use ($tag) {
+            return $fork->getTag() === $tag;
         });
 
         return new ForkCollection($collection);