Merge pull request #1 from arnested/master

Allow function parameters to be unused without provoking unused-var warning

Author: illusori

Sniffs/CodeAnalysis/VariableAnalysisSniff.php

@@ -344,6 +344,12 @@ class Generic_Sniffs_CodeAnalysis_VariableAnalysisSniff implements PHP_CodeSniff
      */
     public $allowUnusedCaughtExceptions = false;
 
+    /**
+     *  Allow function parameters to be unused without provoking unused-var warning.
+     *  Set generic.codeanalysis.variableanalysis.allowUnusedFunctionParameters to a true value.
+     */
+    public $allowUnusedFunctionParameters = false;
+
     /**
      *  A list of names of placeholder variables that you want to ignore from
      *  unused variable warnings, ie things like $junk.
@@ -1453,6 +1459,9 @@ protected function processScopeClose(
             if ($varInfo->ignoreUnused || isset($varInfo->firstRead)) {
                 continue;
             }
+            if ($this->allowUnusedFunctionParameters && $varInfo->scopeType == 'param') {
+                continue;
+            }
             if ($varInfo->passByReference && isset($varInfo->firstInitialized)) {
                 // If we're pass-by-reference then it's a common pattern to
                 // use the variable to return data to the caller, so any