Generic/OpeningFunctionBraceKernighanRitchie: simplify logic to find …

…end of the function declaration

When finding the end of the function declaration, the sniff had an
unnecessary block of code to change the `$end` parameter passed to
`findPrevious()` when handling a closure with a `use` statement.

Since we are looking for the first non-empty token before the opening
curly brace, it is not necessary to change the value of the `$end`
parameter. Actually, we don't even need to limit the search and we can
pass `null` (the default value). The returned first non-empty token will
be the same anyway regardless if `$end` is the closing parenthesis
before `use`, after `use` or `null`.

src/Standards/Generic/Sniffs/Functions/OpeningFunctionBraceKernighanRitchieSniff.php

@@ -72,17 +72,9 @@ public function process(File $phpcsFile, $stackPtr)
         }
 
         $openingBrace = $tokens[$stackPtr]['scope_opener'];
-        $closeBracket = $tokens[$stackPtr]['parenthesis_closer'];
-        if ($tokens[$stackPtr]['code'] === T_CLOSURE) {
-            $use = $phpcsFile->findNext(T_USE, ($closeBracket + 1), $tokens[$stackPtr]['scope_opener']);
-            if ($use !== false) {
-                $openBracket  = $phpcsFile->findNext(T_OPEN_PARENTHESIS, ($use + 1));
-                $closeBracket = $tokens[$openBracket]['parenthesis_closer'];
-            }
-        }
 
         // Find the end of the function declaration.
-        $prev = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($openingBrace - 1), $closeBracket, true);
+        $prev = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($openingBrace - 1), null, true);
 
         $functionLine = $tokens[$prev]['line'];
         $braceLine    = $tokens[$openingBrace]['line'];