nikic / PHP-Parser

grammar/analyze.php

Doc Comments +6 added lines

@@ -20,10 +20,16 @@
     'case_separator', 'optional_comma'
 ));
 
+/**
+ * @param string $regex
+ */
 function regex($regex) {
     return '~' . LIB . '(?:' . str_replace('~', '\~', $regex) . ')~';
 }
 
+/**
+ * @param string $regex
+ */
 function magicSplit($regex, $string) {
     $pieces = preg_split(regex('(?:(?&string)|(?&comment)|(?&code))(*SKIP)(*FAIL)|' . $regex), $string);
 

grammar/rebuildParsers.php

Doc Comments +9 added lines

@@ -175,6 +175,9 @@ 
     );
 }
 
+/**
+ * @param integer $num
+ */
 function assertArgs($num, $args, $name) {
     if ($num != count($args)) {
         die('Wrong argument count for ' . $name . '().');
@@ -203,10 +206,16 @@ 
 /// Regex helper functions ///
 //////////////////////////////
 
+/**
+ * @param string $regex
+ */
 function regex($regex) {
     return '~' . LIB . '(?:' . str_replace('~', '\~', $regex) . ')~';
 }
 
+/**
+ * @param string $regex
+ */
 function magicSplit($regex, $string) {
     $pieces = preg_split(regex('(?:(?&string)|(?&comment)|(?&code))(*SKIP)(*FAIL)|' . $regex), $string);
 

lib/PhpParser/Builder/Class_.php

Doc Comments -2 removed lines

@@ -44,8 +44,6 @@
     /**
      * Implements one or more interfaces.
      *
-     * @param Name|string $interface Name of interface to implement
-     * @param Name|string $...       More interfaces to implement
      *
      * @return $this The builder instance (for fluid interface)
      */

lib/PhpParser/Builder/Interface_.php

Doc Comments -2 removed lines

@@ -25,8 +25,6 @@
     /**
      * Extends one or more interfaces.
      *
-     * @param Name|string $interface Name of interface to extend
-     * @param Name|string $...       More interfaces to extend
      *
      * @return $this The builder instance (for fluid interface)
      */

lib/PhpParser/NodeVisitor/NameResolver.php

Doc Comments +3 added lines

@@ -110,6 +110,9 @@
         );
     }
 
+    /**
+     * @param Name $type
+     */
     protected function addAlias(Stmt\UseUse $use, $type, Name $prefix = null) {
         // Add prefix for group uses
         $name = $prefix ? Name::concat($prefix, $use->name) : $use->name;

lib/PhpParser/Parser/Multiple.php

Doc Comments +3 added lines

@@ -51,6 +51,9 @@
         return $this->errors;
     }
 
+    /**
+     * @param string $code
+     */
     private function tryParse(Parser $parser, $code) {
         $stmts = null;
         $error = null;

lib/PhpParser/PrettyPrinter/Standard.php

Doc Comments +12 added lines

@@ -534,6 +534,9 @@ 
              . ($node->name->getLast() !== $node->alias ? ' as ' . $node->alias : '');
     }
 
+    /**
+     * @param integer $type
+     */
     private function pUseType($type) {
         return $type === Stmt\Use_::TYPE_FUNCTION ? 'function '
             : ($type === Stmt\Use_::TYPE_CONSTANT ? 'const ' : '');
@@ -744,6 +747,9 @@ 
         return is_string($node) ? $node : $this->p($node);
     }
 
+    /**
+     * @param string $afterClassToken
+     */
     protected function pClassCommon(Stmt\Class_ $node, $afterClassToken) {
         return $this->pModifiers($node->type)
         . 'class' . $afterClassToken
@@ -760,6 +766,9 @@ 
         }
     }
 
+    /**
+     * @param integer $modifiers
+     */
     protected function pModifiers($modifiers) {
         return ($modifiers & Stmt\Class_::MODIFIER_PUBLIC    ? 'public '    : '')
              . ($modifiers & Stmt\Class_::MODIFIER_PROTECTED ? 'protected ' : '')
@@ -769,6 +778,9 @@ 
              . ($modifiers & Stmt\Class_::MODIFIER_FINAL     ? 'final '     : '');
     }
 
+    /**
+     * @param string $quote
+     */
     protected function pEncapsList(array $encapsList, $quote) {
         $return = '';
         foreach ($encapsList as $element) {

Spacing +9 added lines, -9 removed lines

@@ -463,7 +463,7 @@ 
         return ($node->static ? 'static ' : '')
              . 'function ' . ($node->byRef ? '&' : '')
              . '(' . $this->pCommaSeparated($node->params) . ')'
-             . (!empty($node->uses) ? ' use(' . $this->pCommaSeparated($node->uses) . ')': '')
+             . (!empty($node->uses) ? ' use(' . $this->pCommaSeparated($node->uses) . ')' : '')
              . (null !== $node->returnType ? ' : ' . $this->pType($node->returnType) : '')
              . ' {' . $this->pStmts($node->stmts) . "\n" . '}';
     }
@@ -570,7 +570,7 @@ 
         return (null !== $node->trait ? $this->p($node->trait) . '::' : '')
              . $node->method . ' as'
              . (null !== $node->newModifier ? ' ' . rtrim($this->pModifiers($node->newModifier), ' ') : '')
-             . (null !== $node->newName     ? ' ' . $node->newName                        : '')
+             . (null !== $node->newName ? ' ' . $node->newName : '')
              . ';';
     }
 
@@ -761,12 +761,12 @@ 
     }
 
     protected function pModifiers($modifiers) {
-        return ($modifiers & Stmt\Class_::MODIFIER_PUBLIC    ? 'public '    : '')
+        return ($modifiers & Stmt\Class_::MODIFIER_PUBLIC ? 'public ' : '')
              . ($modifiers & Stmt\Class_::MODIFIER_PROTECTED ? 'protected ' : '')
-             . ($modifiers & Stmt\Class_::MODIFIER_PRIVATE   ? 'private '   : '')
-             . ($modifiers & Stmt\Class_::MODIFIER_STATIC    ? 'static '    : '')
-             . ($modifiers & Stmt\Class_::MODIFIER_ABSTRACT  ? 'abstract '  : '')
-             . ($modifiers & Stmt\Class_::MODIFIER_FINAL     ? 'final '     : '');
+             . ($modifiers & Stmt\Class_::MODIFIER_PRIVATE ? 'private ' : '')
+             . ($modifiers & Stmt\Class_::MODIFIER_STATIC ? 'static ' : '')
+             . ($modifiers & Stmt\Class_::MODIFIER_ABSTRACT ? 'abstract ' : '')
+             . ($modifiers & Stmt\Class_::MODIFIER_FINAL ? 'final ' : '');
     }
 
     protected function pEncapsList(array $encapsList, $quote) {
@@ -797,7 +797,7 @@ 
             || $node instanceof Expr\ClassConstFetch
         ) {
             return $this->p($node);
-        } else  {
+        } else {
             return '(' . $this->p($node) . ')';
         }
     }
@@ -812,7 +812,7 @@ 
             || $node instanceof Expr\Array_
         ) {
             return $this->p($node);
-        } else  {
+        } else {
             return '(' . $this->p($node) . ')';
         }
     }

lib/PhpParser/PrettyPrinterAbstract.php

Doc Comments +13 added lines, -1 removed lines

@@ -187,6 +187,10 @@ 
         return $this->{'p' . $node->getType()}($node);
     }
 
+    /**
+     * @param string $type
+     * @param string $operatorString
+     */
     protected function pInfixOp($type, Node $leftNode, $operatorString, Node $rightNode) {
         list($precedence, $associativity) = $this->precedenceMap[$type];
 
@@ -195,11 +199,19 @@ 
              . $this->pPrec($rightNode, $precedence, $associativity, 1);
     }
 
+    /**
+     * @param string $type
+     * @param string $operatorString
+     */
     protected function pPrefixOp($type, $operatorString, Node $node) {
         list($precedence, $associativity) = $this->precedenceMap[$type];
         return $operatorString . $this->pPrec($node, $precedence, $associativity, 1);
     }
 
+    /**
+     * @param string $type
+     * @param string $operatorString
+     */
     protected function pPostfixOp($type, Node $node, $operatorString) {
         list($precedence, $associativity) = $this->precedenceMap[$type];
         return $this->pPrec($node, $precedence, $associativity, -1) . $operatorString;
@@ -264,7 +276,7 @@ 
      *
      * @param string $string Not to be indented string
      *
-     * @return mixed String marked with $this->noIndentToken's.
+     * @return string String marked with $this->noIndentToken's.
      */
     protected function pNoIndent($string) {
         return str_replace("\n", "\n" . $this->noIndentToken, $string);

Spacing +53 added lines, -53 removed lines

@@ -9,65 +9,65 @@
 {
     protected $precedenceMap = array(
         // [precedence, associativity] where for the latter -1 is %left, 0 is %nonassoc and 1 is %right
-        'Expr_BinaryOp_Pow'            => array(  0,  1),
-        'Expr_BitwiseNot'              => array( 10,  1),
-        'Expr_PreInc'                  => array( 10,  1),
-        'Expr_PreDec'                  => array( 10,  1),
-        'Expr_PostInc'                 => array( 10, -1),
-        'Expr_PostDec'                 => array( 10, -1),
-        'Expr_UnaryPlus'               => array( 10,  1),
-        'Expr_UnaryMinus'              => array( 10,  1),
-        'Expr_Cast_Int'                => array( 10,  1),
-        'Expr_Cast_Double'             => array( 10,  1),
-        'Expr_Cast_String'             => array( 10,  1),
-        'Expr_Cast_Array'              => array( 10,  1),
-        'Expr_Cast_Object'             => array( 10,  1),
-        'Expr_Cast_Bool'               => array( 10,  1),
-        'Expr_Cast_Unset'              => array( 10,  1),
-        'Expr_ErrorSuppress'           => array( 10,  1),
-        'Expr_Instanceof'              => array( 20,  0),
-        'Expr_BooleanNot'              => array( 30,  1),
-        'Expr_BinaryOp_Mul'            => array( 40, -1),
-        'Expr_BinaryOp_Div'            => array( 40, -1),
-        'Expr_BinaryOp_Mod'            => array( 40, -1),
-        'Expr_BinaryOp_Plus'           => array( 50, -1),
-        'Expr_BinaryOp_Minus'          => array( 50, -1),
-        'Expr_BinaryOp_Concat'         => array( 50, -1),
-        'Expr_BinaryOp_ShiftLeft'      => array( 60, -1),
-        'Expr_BinaryOp_ShiftRight'     => array( 60, -1),
-        'Expr_BinaryOp_Smaller'        => array( 70,  0),
-        'Expr_BinaryOp_SmallerOrEqual' => array( 70,  0),
-        'Expr_BinaryOp_Greater'        => array( 70,  0),
-        'Expr_BinaryOp_GreaterOrEqual' => array( 70,  0),
-        'Expr_BinaryOp_Equal'          => array( 80,  0),
-        'Expr_BinaryOp_NotEqual'       => array( 80,  0),
-        'Expr_BinaryOp_Identical'      => array( 80,  0),
-        'Expr_BinaryOp_NotIdentical'   => array( 80,  0),
-        'Expr_BinaryOp_Spaceship'      => array( 80,  0),
-        'Expr_BinaryOp_BitwiseAnd'     => array( 90, -1),
+        'Expr_BinaryOp_Pow'            => array(0, 1),
+        'Expr_BitwiseNot'              => array(10, 1),
+        'Expr_PreInc'                  => array(10, 1),
+        'Expr_PreDec'                  => array(10, 1),
+        'Expr_PostInc'                 => array(10, -1),
+        'Expr_PostDec'                 => array(10, -1),
+        'Expr_UnaryPlus'               => array(10, 1),
+        'Expr_UnaryMinus'              => array(10, 1),
+        'Expr_Cast_Int'                => array(10, 1),
+        'Expr_Cast_Double'             => array(10, 1),
+        'Expr_Cast_String'             => array(10, 1),
+        'Expr_Cast_Array'              => array(10, 1),
+        'Expr_Cast_Object'             => array(10, 1),
+        'Expr_Cast_Bool'               => array(10, 1),
+        'Expr_Cast_Unset'              => array(10, 1),
+        'Expr_ErrorSuppress'           => array(10, 1),
+        'Expr_Instanceof'              => array(20, 0),
+        'Expr_BooleanNot'              => array(30, 1),
+        'Expr_BinaryOp_Mul'            => array(40, -1),
+        'Expr_BinaryOp_Div'            => array(40, -1),
+        'Expr_BinaryOp_Mod'            => array(40, -1),
+        'Expr_BinaryOp_Plus'           => array(50, -1),
+        'Expr_BinaryOp_Minus'          => array(50, -1),
+        'Expr_BinaryOp_Concat'         => array(50, -1),
+        'Expr_BinaryOp_ShiftLeft'      => array(60, -1),
+        'Expr_BinaryOp_ShiftRight'     => array(60, -1),
+        'Expr_BinaryOp_Smaller'        => array(70, 0),
+        'Expr_BinaryOp_SmallerOrEqual' => array(70, 0),
+        'Expr_BinaryOp_Greater'        => array(70, 0),
+        'Expr_BinaryOp_GreaterOrEqual' => array(70, 0),
+        'Expr_BinaryOp_Equal'          => array(80, 0),
+        'Expr_BinaryOp_NotEqual'       => array(80, 0),
+        'Expr_BinaryOp_Identical'      => array(80, 0),
+        'Expr_BinaryOp_NotIdentical'   => array(80, 0),
+        'Expr_BinaryOp_Spaceship'      => array(80, 0),
+        'Expr_BinaryOp_BitwiseAnd'     => array(90, -1),
         'Expr_BinaryOp_BitwiseXor'     => array(100, -1),
         'Expr_BinaryOp_BitwiseOr'      => array(110, -1),
         'Expr_BinaryOp_BooleanAnd'     => array(120, -1),
         'Expr_BinaryOp_BooleanOr'      => array(130, -1),
-        'Expr_BinaryOp_Coalesce'       => array(140,  1),
+        'Expr_BinaryOp_Coalesce'       => array(140, 1),
         'Expr_Ternary'                 => array(150, -1),
         // parser uses %left for assignments, but they really behave as %right
-        'Expr_Assign'                  => array(160,  1),
-        'Expr_AssignRef'               => array(160,  1),
-        'Expr_AssignOp_Plus'           => array(160,  1),
-        'Expr_AssignOp_Minus'          => array(160,  1),
-        'Expr_AssignOp_Mul'            => array(160,  1),
-        'Expr_AssignOp_Div'            => array(160,  1),
-        'Expr_AssignOp_Concat'         => array(160,  1),
-        'Expr_AssignOp_Mod'            => array(160,  1),
-        'Expr_AssignOp_BitwiseAnd'     => array(160,  1),
-        'Expr_AssignOp_BitwiseOr'      => array(160,  1),
-        'Expr_AssignOp_BitwiseXor'     => array(160,  1),
-        'Expr_AssignOp_ShiftLeft'      => array(160,  1),
-        'Expr_AssignOp_ShiftRight'     => array(160,  1),
-        'Expr_AssignOp_Pow'            => array(160,  1),
-        'Expr_YieldFrom'               => array(165,  1),
-        'Expr_Print'                   => array(168,  1),
+        'Expr_Assign'                  => array(160, 1),
+        'Expr_AssignRef'               => array(160, 1),
+        'Expr_AssignOp_Plus'           => array(160, 1),
+        'Expr_AssignOp_Minus'          => array(160, 1),
+        'Expr_AssignOp_Mul'            => array(160, 1),
+        'Expr_AssignOp_Div'            => array(160, 1),
+        'Expr_AssignOp_Concat'         => array(160, 1),
+        'Expr_AssignOp_Mod'            => array(160, 1),
+        'Expr_AssignOp_BitwiseAnd'     => array(160, 1),
+        'Expr_AssignOp_BitwiseOr'      => array(160, 1),
+        'Expr_AssignOp_BitwiseXor'     => array(160, 1),
+        'Expr_AssignOp_ShiftLeft'      => array(160, 1),
+        'Expr_AssignOp_ShiftRight'     => array(160, 1),
+        'Expr_AssignOp_Pow'            => array(160, 1),
+        'Expr_YieldFrom'               => array(165, 1),
+        'Expr_Print'                   => array(168, 1),
         'Expr_BinaryOp_LogicalAnd'     => array(170, -1),
         'Expr_BinaryOp_LogicalXor'     => array(180, -1),
         'Expr_BinaryOp_LogicalOr'      => array(190, -1),

lib/PhpParser/Lexer.php

Spacing +3 added lines, -3 removed lines

@@ -62,7 +62,7 @@ 
 
         $this->code = $code; // keep the code around for __halt_compiler() handling
         $this->pos  = -1;
-        $this->line =  1;
+        $this->line = 1;
         $this->filePos = 0;
     }
 
@@ -264,10 +264,10 @@ 
             if (T_DOUBLE_COLON === $i) {
                 // T_DOUBLE_COLON is equivalent to T_PAAMAYIM_NEKUDOTAYIM
                 $tokenMap[$i] = Tokens::T_PAAMAYIM_NEKUDOTAYIM;
-            } elseif(T_OPEN_TAG_WITH_ECHO === $i) {
+            } elseif (T_OPEN_TAG_WITH_ECHO === $i) {
                 // T_OPEN_TAG_WITH_ECHO with dropped T_OPEN_TAG results in T_ECHO
                 $tokenMap[$i] = Tokens::T_ECHO;
-            } elseif(T_CLOSE_TAG === $i) {
+            } elseif (T_CLOSE_TAG === $i) {
                 // T_CLOSE_TAG is equivalent to ';'
                 $tokenMap[$i] = ord(';');
             } elseif ('UNKNOWN' !== $name = token_name($i)) {