Symplify / PHP7_CodeSniffer

scripts/build-phar.php

Indentation +1 added lines, -1 removed lines

@@ -292,7 +292,7 @@
     return array(
             'whiz',
             'bang',
-           );
+            );
 
     case 2:
     return helper_func(

src/Files/File.php

Doc Comments +2 added lines, -2 removed lines

@@ -866,7 +866,7 @@ 
      * @param string $error    The error message.
      * @param int    $line     The line on which the error occurred.
      * @param string $code     A violation code unique to the sniff message.
-     * @param array  $data     Replacements for the error message.
+     * @param string[]  $data     Replacements for the error message.
      * @param int    $severity The severity level for this error. A value of 0 will be converted into the default severity level.
      *                          will be converted into the default severity level.
      *
@@ -890,7 +890,7 @@ 
      * @param string $warning  The error message.
      * @param int    $line     The line on which the warning occurred.
      * @param string $code     A violation code unique to the sniff message.
-     * @param array  $data     Replacements for the warning message.
+     * @param string[]  $data     Replacements for the warning message.
      * @param int    $severity The severity level for this warning. A value of 0 will be converted into the default severity level.
      *                          will be converted into the default severity level.
      *

Indentation +56 added lines, -56 removed lines

@@ -780,11 +780,11 @@ 
             }
 
             $checkCodes = array(
-                           $sniffCode,
-                           $parts[0].'.'.$parts[1].'.'.$parts[2],
-                           $parts[0].'.'.$parts[1],
-                           $parts[0],
-                          );
+                            $sniffCode,
+                            $parts[0].'.'.$parts[1].'.'.$parts[2],
+                            $parts[0].'.'.$parts[1],
+                            $parts[0],
+                            );
         }//end if
 
         // Filter out any messages for sniffs that shouldn't have run
@@ -860,8 +860,8 @@ 
                 // While there is support for a type of each pattern
                 // (absolute or relative) we don't actually support it here.
                 $replacements = array(
-                                 '\\,' => ',',
-                                 '*'   => '.*',
+                                    '\\,' => ',',
+                                    '*'   => '.*',
                                 );
 
                 // We assume a / directory separator, as do the exclude rules
@@ -908,12 +908,12 @@ 
         }
 
         $messages[$line][$column][] = array(
-                                       'message'  => $message,
-                                       'source'   => $sniffCode,
-                                       'listener' => $this->activeListener,
-                                       'severity' => $severity,
-                                       'fixable'  => $fixable,
-                                      );
+                                        'message'  => $message,
+                                        'source'   => $sniffCode,
+                                        'listener' => $this->activeListener,
+                                        'severity' => $severity,
+                                        'fixable'  => $fixable,
+                                        );
 
         if (PHP_CODESNIFFER_VERBOSITY > 1
             && $this->fixer->enabled === true
@@ -1328,16 +1328,16 @@ 
         }
 
         $valid = array(
-                  T_PUBLIC      => T_PUBLIC,
-                  T_PRIVATE     => T_PRIVATE,
-                  T_PROTECTED   => T_PROTECTED,
-                  T_STATIC      => T_STATIC,
-                  T_FINAL       => T_FINAL,
-                  T_ABSTRACT    => T_ABSTRACT,
-                  T_WHITESPACE  => T_WHITESPACE,
-                  T_COMMENT     => T_COMMENT,
-                  T_DOC_COMMENT => T_DOC_COMMENT,
-                 );
+                    T_PUBLIC      => T_PUBLIC,
+                    T_PRIVATE     => T_PRIVATE,
+                    T_PROTECTED   => T_PROTECTED,
+                    T_STATIC      => T_STATIC,
+                    T_FINAL       => T_FINAL,
+                    T_ABSTRACT    => T_ABSTRACT,
+                    T_WHITESPACE  => T_WHITESPACE,
+                    T_COMMENT     => T_COMMENT,
+                    T_DOC_COMMENT => T_DOC_COMMENT,
+                    );
 
         $scope          = 'public';
         $scopeSpecified = false;
@@ -1383,7 +1383,7 @@ 
                 'is_final'        => $isFinal,
                 'is_static'       => $isStatic,
                 'is_closure'      => $isClosure,
-               );
+                );
 
     }//end getMethodProperties()
 
@@ -1441,16 +1441,16 @@ 
         }
 
         $valid = array(
-                  T_PUBLIC      => T_PUBLIC,
-                  T_PRIVATE     => T_PRIVATE,
-                  T_PROTECTED   => T_PROTECTED,
-                  T_STATIC      => T_STATIC,
-                  T_WHITESPACE  => T_WHITESPACE,
-                  T_COMMENT     => T_COMMENT,
-                  T_DOC_COMMENT => T_DOC_COMMENT,
-                  T_VARIABLE    => T_VARIABLE,
-                  T_COMMA       => T_COMMA,
-                 );
+                    T_PUBLIC      => T_PUBLIC,
+                    T_PRIVATE     => T_PRIVATE,
+                    T_PROTECTED   => T_PROTECTED,
+                    T_STATIC      => T_STATIC,
+                    T_WHITESPACE  => T_WHITESPACE,
+                    T_COMMENT     => T_COMMENT,
+                    T_DOC_COMMENT => T_DOC_COMMENT,
+                    T_VARIABLE    => T_VARIABLE,
+                    T_COMMA       => T_COMMA,
+                    );
 
         $scope          = 'public';
         $scopeSpecified = false;
@@ -1484,7 +1484,7 @@ 
                 'scope'           => $scope,
                 'scope_specified' => $scopeSpecified,
                 'is_static'       => $isStatic,
-               );
+                );
 
     }//end getMemberProperties()
 
@@ -1514,12 +1514,12 @@ 
         }
 
         $valid = array(
-                  T_FINAL       => T_FINAL,
-                  T_ABSTRACT    => T_ABSTRACT,
-                  T_WHITESPACE  => T_WHITESPACE,
-                  T_COMMENT     => T_COMMENT,
-                  T_DOC_COMMENT => T_DOC_COMMENT,
-                 );
+                    T_FINAL       => T_FINAL,
+                    T_ABSTRACT    => T_ABSTRACT,
+                    T_WHITESPACE  => T_WHITESPACE,
+                    T_COMMENT     => T_COMMENT,
+                    T_DOC_COMMENT => T_DOC_COMMENT,
+                    );
 
         $isAbstract = false;
         $isFinal    = false;
@@ -1543,7 +1543,7 @@ 
         return array(
                 'is_abstract' => $isAbstract,
                 'is_final'    => $isFinal,
-               );
+                );
 
     }//end getClassProperties()
 
@@ -1893,17 +1893,17 @@ 
     public function findEndOfStatement($start, $ignore=null)
     {
         $endTokens = array(
-                      T_COLON                => true,
-                      T_COMMA                => true,
-                      T_DOUBLE_ARROW         => true,
-                      T_SEMICOLON            => true,
-                      T_CLOSE_PARENTHESIS    => true,
-                      T_CLOSE_SQUARE_BRACKET => true,
-                      T_CLOSE_CURLY_BRACKET  => true,
-                      T_CLOSE_SHORT_ARRAY    => true,
-                      T_OPEN_TAG             => true,
-                      T_CLOSE_TAG            => true,
-                     );
+                        T_COLON                => true,
+                        T_COMMA                => true,
+                        T_DOUBLE_ARROW         => true,
+                        T_SEMICOLON            => true,
+                        T_CLOSE_PARENTHESIS    => true,
+                        T_CLOSE_SQUARE_BRACKET => true,
+                        T_CLOSE_CURLY_BRACKET  => true,
+                        T_CLOSE_SHORT_ARRAY    => true,
+                        T_OPEN_TAG             => true,
+                        T_CLOSE_TAG            => true,
+                        );
 
         if ($ignore !== null) {
             $ignore = (array) $ignore;
@@ -2116,9 +2116,9 @@ 
         }
 
         $find = array(
-                 T_NS_SEPARATOR,
-                 T_STRING,
-                 T_WHITESPACE,
+                    T_NS_SEPARATOR,
+                    T_STRING,
+                    T_WHITESPACE,
                 );
 
         $end  = $this->findNext($find, ($extendsIndex + 1), $classCloserIndex, true);

Switch Indentation +112 added lines, -112 removed lines

@@ -1209,88 +1209,88 @@ 
             }
 
             switch ($this->tokens[$i]['code']) {
-            case T_BITWISE_AND:
-                $passByReference = true;
-                break;
-            case T_VARIABLE:
-                $currVar = $i;
-                break;
-            case T_ELLIPSIS:
-                $variableLength = true;
-                break;
-            case T_ARRAY_HINT:
-            case T_CALLABLE:
-                $typeHint = $this->tokens[$i]['content'];
-                break;
-            case T_STRING:
-                // This is a string, so it may be a type hint, but it could
-                // also be a constant used as a default value.
-                $prevComma = false;
-                for ($t = $i; $t >= $opener; $t--) {
-                    if ($this->tokens[$t]['code'] === T_COMMA) {
-                        $prevComma = $t;
-                        break;
+                case T_BITWISE_AND:
+                    $passByReference = true;
+                    break;
+                case T_VARIABLE:
+                    $currVar = $i;
+                    break;
+                case T_ELLIPSIS:
+                    $variableLength = true;
+                    break;
+                case T_ARRAY_HINT:
+                case T_CALLABLE:
+                    $typeHint = $this->tokens[$i]['content'];
+                    break;
+                case T_STRING:
+                    // This is a string, so it may be a type hint, but it could
+                    // also be a constant used as a default value.
+                    $prevComma = false;
+                    for ($t = $i; $t >= $opener; $t--) {
+                        if ($this->tokens[$t]['code'] === T_COMMA) {
+                            $prevComma = $t;
+                            break;
+                        }
                     }
-                }
 
-                if ($prevComma !== false) {
-                    $nextEquals = false;
-                    for ($t = $prevComma; $t < $i; $t++) {
-                        if ($this->tokens[$t]['code'] === T_EQUAL) {
-                            $nextEquals = $t;
+                    if ($prevComma !== false) {
+                        $nextEquals = false;
+                        for ($t = $prevComma; $t < $i; $t++) {
+                            if ($this->tokens[$t]['code'] === T_EQUAL) {
+                                $nextEquals = $t;
+                                break;
+                            }
+                        }
+
+                        if ($nextEquals !== false) {
                             break;
                         }
                     }
 
-                    if ($nextEquals !== false) {
-                        break;
+                    if ($defaultStart === null) {
+                        $typeHint .= $this->tokens[$i]['content'];
+                    }
+                    break;
+                case T_NS_SEPARATOR:
+                    // Part of a type hint or default value.
+                    if ($defaultStart === null) {
+                        $typeHint .= $this->tokens[$i]['content'];
+                    }
+                    break;
+                case T_CLOSE_PARENTHESIS:
+                case T_COMMA:
+                    // If it's null, then there must be no parameters for this
+                    // method.
+                    if ($currVar === null) {
+                        continue;
                     }
-                }
-
-                if ($defaultStart === null) {
-                    $typeHint .= $this->tokens[$i]['content'];
-                }
-                break;
-            case T_NS_SEPARATOR:
-                // Part of a type hint or default value.
-                if ($defaultStart === null) {
-                    $typeHint .= $this->tokens[$i]['content'];
-                }
-                break;
-            case T_CLOSE_PARENTHESIS:
-            case T_COMMA:
-                // If it's null, then there must be no parameters for this
-                // method.
-                if ($currVar === null) {
-                    continue;
-                }
 
-                $vars[$paramCount]         = array();
-                $vars[$paramCount]['name'] = $this->tokens[$currVar]['content'];
+                    $vars[$paramCount]         = array();
+                    $vars[$paramCount]['name'] = $this->tokens[$currVar]['content'];
 
-                if ($defaultStart !== null) {
-                    $vars[$paramCount]['default']
-                        = $this->getTokensAsString(
-                            $defaultStart,
-                            ($i - $defaultStart)
-                        );
-                }
+                    if ($defaultStart !== null) {
+                        $vars[$paramCount]['default']
+                            = $this->getTokensAsString(
+                                $defaultStart,
+                                ($i - $defaultStart)
+                            );
+                    }
 
-                $vars[$paramCount]['pass_by_reference'] = $passByReference;
-                $vars[$paramCount]['variable_length']   = $variableLength;
-                $vars[$paramCount]['type_hint']         = $typeHint;
+                    $vars[$paramCount]['pass_by_reference'] = $passByReference;
+                    $vars[$paramCount]['variable_length']   = $variableLength;
+                    $vars[$paramCount]['type_hint']         = $typeHint;
 
-                // Reset the vars, as we are about to process the next parameter.
-                $defaultStart    = null;
-                $passByReference = false;
-                $variableLength  = false;
-                $typeHint        = '';
+                    // Reset the vars, as we are about to process the next parameter.
+                    $defaultStart    = null;
+                    $passByReference = false;
+                    $variableLength  = false;
+                    $typeHint        = '';
 
-                $paramCount++;
-                break;
-            case T_EQUAL:
-                $defaultStart = ($i + 1);
-                break;
+                    $paramCount++;
+                    break;
+                case T_EQUAL:
+                    $defaultStart = ($i + 1);
+                    break;
             }//end switch
         }//end for
 
@@ -1352,27 +1352,27 @@ 
             }
 
             switch ($this->tokens[$i]['code']) {
-            case T_PUBLIC:
-                $scope          = 'public';
-                $scopeSpecified = true;
-                break;
-            case T_PRIVATE:
-                $scope          = 'private';
-                $scopeSpecified = true;
-                break;
-            case T_PROTECTED:
-                $scope          = 'protected';
-                $scopeSpecified = true;
-                break;
-            case T_ABSTRACT:
-                $isAbstract = true;
-                break;
-            case T_FINAL:
-                $isFinal = true;
-                break;
-            case T_STATIC:
-                $isStatic = true;
-                break;
+                case T_PUBLIC:
+                    $scope          = 'public';
+                    $scopeSpecified = true;
+                    break;
+                case T_PRIVATE:
+                    $scope          = 'private';
+                    $scopeSpecified = true;
+                    break;
+                case T_PROTECTED:
+                    $scope          = 'protected';
+                    $scopeSpecified = true;
+                    break;
+                case T_ABSTRACT:
+                    $isAbstract = true;
+                    break;
+                case T_FINAL:
+                    $isFinal = true;
+                    break;
+                case T_STATIC:
+                    $isStatic = true;
+                    break;
             }//end switch
         }//end for
 
@@ -1462,21 +1462,21 @@ 
             }
 
             switch ($this->tokens[$i]['code']) {
-            case T_PUBLIC:
-                $scope          = 'public';
-                $scopeSpecified = true;
-                break;
-            case T_PRIVATE:
-                $scope          = 'private';
-                $scopeSpecified = true;
-                break;
-            case T_PROTECTED:
-                $scope          = 'protected';
-                $scopeSpecified = true;
-                break;
-            case T_STATIC:
-                $isStatic = true;
-                break;
+                case T_PUBLIC:
+                    $scope          = 'public';
+                    $scopeSpecified = true;
+                    break;
+                case T_PRIVATE:
+                    $scope          = 'private';
+                    $scopeSpecified = true;
+                    break;
+                case T_PROTECTED:
+                    $scope          = 'protected';
+                    $scopeSpecified = true;
+                    break;
+                case T_STATIC:
+                    $isStatic = true;
+                    break;
             }
         }//end for
 
@@ -1530,13 +1530,13 @@ 
             }
 
             switch ($this->tokens[$i]['code']) {
-            case T_ABSTRACT:
-                $isAbstract = true;
-                break;
+                case T_ABSTRACT:
+                    $isAbstract = true;
+                    break;
 
-            case T_FINAL:
-                $isFinal = true;
-                break;
+                case T_FINAL:
+                    $isFinal = true;
+                    break;
             }
         }//end for
 

Spacing +25 added lines, -25 removed lines

@@ -572,9 +572,9 @@ 
         $error,
         $stackPtr,
         $code,
-        $data=array(),
-        $severity=0,
-        $fixable=false
+        $data = array(),
+        $severity = 0,
+        $fixable = false
     ) {
         if ($stackPtr === null) {
             $line   = 1;
@@ -606,9 +606,9 @@ 
         $warning,
         $stackPtr,
         $code,
-        $data=array(),
-        $severity=0,
-        $fixable=false
+        $data = array(),
+        $severity = 0,
+        $fixable = false
     ) {
         if ($stackPtr === null) {
             $line   = 1;
@@ -639,8 +639,8 @@ 
         $error,
         $line,
         $code,
-        $data=array(),
-        $severity=0
+        $data = array(),
+        $severity = 0
     ) {
         return $this->addMessage(true, $error, $line, 1, $code, $data, $severity, false);
 
@@ -663,8 +663,8 @@ 
         $warning,
         $line,
         $code,
-        $data=array(),
-        $severity=0
+        $data = array(),
+        $severity = 0
     ) {
         return $this->addMessage(false, $warning, $line, 1, $code, $data, $severity, false);
 
@@ -689,8 +689,8 @@ 
         $error,
         $stackPtr,
         $code,
-        $data=array(),
-        $severity=0
+        $data = array(),
+        $severity = 0
     ) {
         $recorded = $this->addError($error, $stackPtr, $code, $data, $severity, true);
         if ($recorded === true && $this->fixer->enabled === true) {
@@ -720,8 +720,8 @@ 
         $warning,
         $stackPtr,
         $code,
-        $data=array(),
-        $severity=0
+        $data = array(),
+        $severity = 0
     ) {
         $recorded = $this->addWarning($warning, $stackPtr, $code, $data, $severity, true);
         if ($recorded === true && $this->fixer->enabled === true) {
@@ -1698,10 +1698,10 @@ 
     public function findPrevious(
         $types,
         $start,
-        $end=null,
-        $exclude=false,
-        $value=null,
-        $local=false
+        $end = null,
+        $exclude = false,
+        $value = null,
+        $local = false
     ) {
         $types = (array) $types;
 
@@ -1779,10 +1779,10 @@ 
     public function findNext(
         $types,
         $start,
-        $end=null,
-        $exclude=false,
-        $value=null,
-        $local=false
+        $end = null,
+        $exclude = false,
+        $value = null,
+        $local = false
     ) {
         $types = (array) $types;
 
@@ -1825,7 +1825,7 @@ 
      *
      * @return int
      */
-    public function findStartOfStatement($start, $ignore=null)
+    public function findStartOfStatement($start, $ignore = null)
     {
         $endTokens = Util\Tokens::$blockOpeners;
 
@@ -1890,7 +1890,7 @@ 
      *
      * @return int
      */
-    public function findEndOfStatement($start, $ignore=null)
+    public function findEndOfStatement($start, $ignore = null)
     {
         $endTokens = array(
                       T_COLON                => true,
@@ -1974,7 +1974,7 @@ 
      *
      * @return int | bool
      */
-    public function findFirstOnLine($types, $start, $exclude=false, $value=null)
+    public function findFirstOnLine($types, $start, $exclude = false, $value = null)
     {
         if (is_array($types) === false) {
             $types = array($types);

src/Files/FileList.php

Doc Comments +1 added lines, -1 removed lines

@@ -193,7 +193,7 @@
     /**
      * Return the file path of the current file being processed.
      *
-     * @return void
+     * @return string
      */
     function key()
     {

Spacing +1 added lines, -1 removed lines

@@ -106,7 +106,7 @@
      *
      * @return void
      */
-    public function addFile($path, $file=null)
+    public function addFile($path, $file = null)
     {
         // No filtering is done for STDIN.
         if ($path === 'STDIN'

src/Fixer.php

Doc Comments +2 added lines, -2 removed lines

@@ -356,7 +356,7 @@ 
     /**
      * Start recording actions for a changeset.
      *
-     * @return void
+     * @return false|null
      */
     public function beginChangeset()
     {
@@ -383,7 +383,7 @@ 
     /**
      * Stop recording actions for a changeset, and apply logged changes.
      *
-     * @return boolean
+     * @return false|null
      */
     public function endChangeset()
     {

Indentation +3 added lines, -3 removed lines

@@ -529,9 +529,9 @@
 
         if (isset($this->_oldTokenValues[$stackPtr]) === false) {
             $this->_oldTokenValues[$stackPtr] = array(
-                                                 'curr' => $content,
-                                                 'prev' => $this->_tokens[$stackPtr],
-                                                 'loop' => $this->loops,
+                                                    'curr' => $content,
+                                                    'prev' => $this->_tokens[$stackPtr],
+                                                    'loop' => $this->loops,
                                                 );
         } else {
             if ($this->_oldTokenValues[$stackPtr]['prev'] === $content

Switch Indentation +8 added lines, -8 removed lines

@@ -283,14 +283,14 @@
         foreach ($diffLines as $line) {
             if (isset($line[0]) === true) {
                 switch ($line[0]) {
-                case '-':
-                    $diff[] = "\033[31m$line\033[0m";
-                    break;
-                case '+':
-                    $diff[] = "\033[32m$line\033[0m";
-                    break;
-                default:
-                    $diff[] = $line;
+                    case '-':
+                        $diff[] = "\033[31m$line\033[0m";
+                        break;
+                    case '+':
+                        $diff[] = "\033[32m$line\033[0m";
+                        break;
+                    default:
+                        $diff[] = $line;
                 }
             }
         }

Spacing +2 added lines, -2 removed lines

@@ -237,7 +237,7 @@ 
      *
      * @return string
      */
-    public function generateDiff($filePath=null, $colors=true)
+    public function generateDiff($filePath = null, $colors = true)
     {
         if ($filePath === null) {
             $filePath = $this->_currentFile->getFilename();
@@ -654,7 +654,7 @@ 
      *
      * @return bool If the change was accepted.
      */
-    public function substrToken($stackPtr, $start, $length=null)
+    public function substrToken($stackPtr, $start, $length = null)
     {
         $current = $this->getTokenContent($stackPtr);
 

src/Generators/Generator.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 
 use PHP_CodeSniffer\Ruleset;
 use PHP_CodeSniffer\Autoload;
-use PHP_CodeSniffer\Util\Common;
 
 abstract class Generator
 {

src/Reports/Cbf.php

Doc Comments +2 added lines, -2 removed lines

@@ -27,11 +27,11 @@
      * its data should be counted in the grand totals.
      *
      * @param array                 $report      Prepared report data.
-     * @param \PHP_CodeSniffer\File $phpcsFile   The file being reported on.
+     * @param File $phpcsFile   The file being reported on.
      * @param bool                  $showSources Show sources?
      * @param int                   $width       Maximum allowed line width.
      *
-     * @return bool
+     * @return null|boolean
      */
     public function generateFileReport($report, File $phpcsFile, $showSources=false, $width=80)
     {

Spacing +5 added lines, -5 removed lines

@@ -33,7 +33,7 @@ 
      *
      * @return bool
      */
-    public function generateFileReport($report, File $phpcsFile, $showSources=false, $width=80)
+    public function generateFileReport($report, File $phpcsFile, $showSources = false, $width = 80)
     {
         $errors = $phpcsFile->getFixableCount();
         if ($errors !== 0) {
@@ -122,10 +122,10 @@ 
         $totalErrors,
         $totalWarnings,
         $totalFixable,
-        $showSources=false,
-        $width=80,
-        $interactive=false,
-        $toScreen=true
+        $showSources = false,
+        $width = 80,
+        $interactive = false,
+        $toScreen = true
     ) {
         $fixed = 0;
         $fails = 0;

src/Reports/Checkstyle.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
      * its data should be counted in the grand totals.
      *
      * @param array                 $report      Prepared report data.
-     * @param \PHP_CodeSniffer\File $phpcsFile   The file being reported on.
+     * @param File $phpcsFile   The file being reported on.
      * @param bool                  $showSources Show sources?
      * @param int                   $width       Maximum allowed line width.
      *

Spacing +5 added lines, -5 removed lines

@@ -30,7 +30,7 @@ 
      *
      * @return bool
      */
-    public function generateFileReport($report, File $phpcsFile, $showSources=false, $width=80)
+    public function generateFileReport($report, File $phpcsFile, $showSources = false, $width = 80)
     {
         $out = new \XMLWriter;
         $out->openMemory();
@@ -93,10 +93,10 @@ 
         $totalErrors,
         $totalWarnings,
         $totalFixable,
-        $showSources=false,
-        $width=80,
-        $interactive=false,
-        $toScreen=true
+        $showSources = false,
+        $width = 80,
+        $interactive = false,
+        $toScreen = true
     ) {
         echo '<?xml version="1.0" encoding="UTF-8"?>'.PHP_EOL;
         echo '<checkstyle version="'.Config::VERSION.'">'.PHP_EOL;

src/Reports/Code.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
      * its data should be counted in the grand totals.
      *
      * @param array                 $report      Prepared report data.
-     * @param \PHP_CodeSniffer\File $phpcsFile   The file being reported on.
+     * @param File $phpcsFile   The file being reported on.
      * @param bool                  $showSources Show sources?
      * @param int                   $width       Maximum allowed line width.
      *

Indentation +3 added lines, -3 removed lines

@@ -79,9 +79,9 @@
 
                 $lastLine++;
                 $lineTokens[$lastLine] = array(
-                                          'start' => $stackPtr,
-                                          'end'   => null,
-                                         );
+                                            'start' => $stackPtr,
+                                            'end'   => null,
+                                            );
             }
         }
 

Spacing +5 added lines, -5 removed lines

@@ -30,7 +30,7 @@ 
      *
      * @return bool
      */
-    public function generateFileReport($report, File $phpcsFile, $showSources=false, $width=80)
+    public function generateFileReport($report, File $phpcsFile, $showSources = false, $width = 80)
     {
         if ($report['errors'] === 0 && $report['warnings'] === 0) {
             // Nothing to print.
@@ -333,10 +333,10 @@ 
         $totalErrors,
         $totalWarnings,
         $totalFixable,
-        $showSources=false,
-        $width=80,
-        $interactive=false,
-        $toScreen=true
+        $showSources = false,
+        $width = 80,
+        $interactive = false,
+        $toScreen = true
     ) {
         if ($cachedData === '') {
             return;

src/Reports/Diff.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
      * its data should be counted in the grand totals.
      *
      * @param array                 $report      Prepared report data.
-     * @param \PHP_CodeSniffer\File $phpcsFile   The file being reported on.
+     * @param File $phpcsFile   The file being reported on.
      * @param bool                  $showSources Show sources?
      * @param int                   $width       Maximum allowed line width.
      *

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 namespace PHP_CodeSniffer\Reports;
 
 use PHP_CodeSniffer\Files\File;
-use PHP_CodeSniffer\Util;
 
 class Diff implements Report
 {

Spacing +5 added lines, -5 removed lines

@@ -30,7 +30,7 @@ 
      *
      * @return bool
      */
-    public function generateFileReport($report, File $phpcsFile, $showSources=false, $width=80)
+    public function generateFileReport($report, File $phpcsFile, $showSources = false, $width = 80)
     {
         $errors = $phpcsFile->getFixableCount();
         if ($errors === 0) {
@@ -146,10 +146,10 @@ 
         $totalErrors,
         $totalWarnings,
         $totalFixable,
-        $showSources=false,
-        $width=80,
-        $interactive=false,
-        $toScreen=true
+        $showSources = false,
+        $width = 80,
+        $interactive = false,
+        $toScreen = true
     ) {
         echo $cachedData;
         if ($toScreen === true) {