stenin-nikita / PHP-Yacc

src/CodeGen/Template.php

Doc Comments +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
 
     /**
      * @param CompressResult $result
-     * @param $resultFile
+     * @param resource $resultFile
      * @param null $headerFile
      *
      * @throws LogicException

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -431,7 +431,7 @@
             case 'YYUNEXPECTED':
                 return \sprintf('%d', Compress::UNEXPECTED);
             case 'YYDEFAULT':
-                return \sprintf('%d', Compress::DEFAULT);
+                return \sprintf('%d', Compress::default);
             case 'YYMAXLEX':
                 return \sprintf('%d', \count($this->compress->yytranslate));
             case 'YYLAST':

Spacing +1 added lines, -1 removed lines

@@ -284,7 +284,7 @@
      */
     protected function skipIf(string $spec): bool
     {
-        [ $dump, $test ] = \explode(' ', $spec, 2);
+        [$dump, $test] = \explode(' ', $spec, 2);
 
         $test = \trim($test);
         switch ($test) {

src/Grammar/Context.php

Doc Comments +1 added lines, -1 removed lines

@@ -219,7 +219,7 @@
      * Context constructor.
      *
      * @param string        $filename
-     * @param resource|null $debugFile
+     * @param null|resource $debugFile
      */
     public function __construct(string $filename = 'YY', resource $debugFile = null)
     {

Spacing +1 added lines, -1 removed lines

@@ -317,7 +317,7 @@
             $nil->code = $code++;
         }
 
-        \usort($this->_symbols, function ($a, $b) {
+        \usort($this->_symbols, function($a, $b) {
             return $a->code <=> $b->code;
         });
     }

src/Grammar/Symbol.php

Doc Comments +2 added lines, -2 removed lines

@@ -79,7 +79,7 @@ 
      * @param int         $terminal
      * @param int         $precedence
      * @param int         $associativity
-     * @param Symbol|null $type
+     * @param null|\self $type
      *
      * @throws LogicException
      */
@@ -174,7 +174,7 @@ 
     }
 
     /**
-     * @param Symbol|null $type
+     * @param null|\self $type
      */
     public function setType(self $type = null)
     {

src/Lalr/Conflict.php

Doc Comments +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
      * Conflict constructor.
      *
      * @param Symbol        $symbol
-     * @param Conflict|null $next
+     * @param null|\self $next
      */
     protected function __construct(Symbol $symbol, self $next = null)
     {
@@ -71,7 +71,7 @@ 
     }
 
     /**
-     * @param Conflict|null $next
+     * @param null|\self $next
      */
     public function setNext(self $next = null)
     {

src/Yacc/Parser.php

Doc Comments +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
     /**
      * @param array $symbols
      * @param int   $n
-     * @param $delm
+     * @param string $delm
      * @param array $attribute
      *
      * @throws ParseException

Spacing +1 added lines, -1 removed lines

@@ -124,7 +124,7 @@
 
         $expanded = $this->macros->apply($this->context, $symbols, $tokens, $n, $attribute);
 
-        $action = \implode('', \array_map(function (Token $token) {
+        $action = \implode('', \array_map(function(Token $token) {
             return $token->getValue();
         }, $expanded));
 

src/Lalr/Generator.php

Upper-Lower-Casing +4 added lines, -4 removed lines

@@ -564,8 +564,8 @@ 
                 /** @var Symbol $left */
                 $left = $gram->body[0];
                 $right = $gram->body[1] ?? null;
-                if (($right === null || ($right->associativity & Production::EMPTY)) && !($left->associativity & Production::EMPTY)) {
-                    $left->setAssociativityFlag(Production::EMPTY);
+                if (($right === null || ($right->associativity & Production::empty)) && !($left->associativity & Production::empty)) {
+                    $left->setAssociativityFlag(Production::empty);
                     $changed = true;
                 }
             }
@@ -574,7 +574,7 @@ 
         if ($this->context->debug) {
             $this->context->debug('EMPTY nonterminals: ');
             foreach ($this->context->nonterminals as $symbol) {
-                if ($symbol->associativity & Production::EMPTY) {
+                if ($symbol->associativity & Production::empty) {
                     $this->context->debug(' '.$symbol->name);
                 }
             }
@@ -681,7 +681,7 @@ 
      */
     protected function findEmpty(Lr1 $tail, Symbol $x): Lr1
     {
-        if (!$this->visited[$x->code] && ($x->associativity & Production::EMPTY)) {
+        if (!$this->visited[$x->code] && ($x->associativity & Production::empty)) {
             $this->visited[$x->code] = true;
 
             /** @var Production $gram */

Doc Comments +1 added lines, -1 removed lines

@@ -434,7 +434,7 @@
      *
      * @throws LogicException
      *
-     * @return int|mixed
+     * @return integer
      */
     protected function comparePrecedence(Production $gram, Symbol $x)
     {

Spacing +5 added lines, -5 removed lines

@@ -176,7 +176,7 @@ 
                 }
             }
 
-            $tmpList = $this->sortList($tmpList, function (Lr1 $x, Lr1 $y) {
+            $tmpList = $this->sortList($tmpList, function(Lr1 $x, Lr1 $y) {
                 $gx = $x->item[-1]->code;
                 $gy = $y->item[-1]->code;
                 if ($gx !== $gy) {
@@ -357,7 +357,7 @@ 
             if (!$tdefact) {
                 $tdefact = -1;
 
-                Utils::stableSort($tmpr, function (Reduce $x, Reduce $y) {
+                Utils::stableSort($tmpr, function(Reduce $x, Reduce $y) {
                     if ($x->number != $y->number) {
                         return $y->number - $x->number;
                     }
@@ -381,11 +381,11 @@ 
             }
 
             // Squeeze tmpr
-            $tmpr = \array_filter($tmpr, function (Reduce $reduce) use ($tdefact) {
+            $tmpr = \array_filter($tmpr, function(Reduce $reduce) use ($tdefact) {
                 return $reduce->number !== $tdefact;
             });
 
-            Utils::stableSort($tmpr, function (Reduce $x, Reduce $y) {
+            Utils::stableSort($tmpr, function(Reduce $x, Reduce $y) {
                 if ($x->symbol !== $y->symbol) {
                     return $x->symbol->code - $y->symbol->code;
                 }
@@ -424,7 +424,7 @@ 
         // do not move initial state
         $initState = \array_shift($this->states);
 
-        Utils::stableSort($this->states, function (State $p, State $q) {
+        Utils::stableSort($this->states, function(State $p, State $q) {
             $numReduces = count($p->reduce) - 1; // -1 for default action
             $pt = $numReduces;
             $pn = count($p->shifts) + $numReduces;

src/Yacc/Production.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
  */
 class Production
 {
-    const EMPTY = 0x10;
+    const empty = 0x10;
 
     /**
      * @var Production|null

src/Lalr/Item.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
     public function offsetGet($index)
     {
         if (!$this->offsetExists($index)) {
-            throw new LogicException("Offset $index does not exist");
+            throw new LogicException("offset $index does not exist");
         }
 
         return $this->production->body[$index + $this->pos];

src/Compress/Compress.php

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
 class Compress
 {
     const UNEXPECTED = 32767;
-    const DEFAULT = -32766;
+    const default = -32766;
     const VACANT = -32768;
 
     /**
@@ -480,7 +480,7 @@ 
                         if ($cTermAction[$i][$j] === $table[$this->context->oTermIndex[$j]]) {
                             $cTermAction[$i][$j] = self::VACANT;
                         } elseif ($cTermAction[$i][$j] === self::VACANT) {
-                            $cTermAction[$i][$j] = self::DEFAULT;
+                            $cTermAction[$i][$j] = self::default;
                         }
                     }
                 }

Spacing +1 added lines, -1 removed lines

@@ -332,7 +332,7 @@
             $this->context->debug("Used aux table:\n");
         }
         $this->context->countAux = $this->context->countClasses;
-        for (; ;) {
+        for (;;) {
             $maxGain = 0;
             $maxAux = null;
             $pre = null;