Generic/CallTimePassByReference: remove unnecessary condition

This commit removes the `|| $prevCode === T_CLASS` condition from a if
statement. The removed check is not necessary as the next thing the
sniff does is to check for an opening parenthesis after `$stackPtr` and
bow out if one is not found. Class definitions will never have an
opening parentesis after its name.

Anonymous classes do have a opening parenthesis after the T_CLASS token.
They are currently not handled correctly by this sniff and this will be
addressed on another commit.

Author: rodrigoprimo

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

@@ -50,12 +50,12 @@ public function process(File $phpcsFile, $stackPtr)
 
         $prev = $phpcsFile->findPrevious($findTokens, ($stackPtr - 1), null, true);
 
-        // Skip tokens that are the names of functions or classes
+        // Skip tokens that are the names of functions
         // within their definitions. For example: function myFunction...
         // "myFunction" is T_STRING but we should skip because it is not a
         // function or method *call*.
         $prevCode = $tokens[$prev]['code'];
-        if ($prevCode === T_FUNCTION || $prevCode === T_CLASS) {
+        if ($prevCode === T_FUNCTION) {
             return;
         }