Security/EscapeOutput: group common token check together

At various points within the `check_code_is_escaped()` loop, the sniff would check whether the current token is a token which can be ignored.

This commit just groups those simple checks, which are not dependent on other logic in the loop, together and places them at the top of the loop for the highest efficiency.

Author: jrfnl

WordPress/Sniffs/Security/EscapeOutputSniff.php

@@ -485,8 +485,12 @@ protected function check_code_is_escaped( $start, $end ) {
 				continue;
 			}
 
-			// Ignore namespace separators.
-			if ( \T_NS_SEPARATOR === $this->tokens[ $i ]['code'] ) {
+			// Skip over irrelevant tokens.
+			if ( isset( Tokens::$magicConstants[ $this->tokens[ $i ]['code'] ] ) // Magic constants for debug functions.
+				|| \T_NS_SEPARATOR === $this->tokens[ $i ]['code']
+				|| \T_DOUBLE_ARROW === $this->tokens[ $i ]['code']
+				|| \T_CLOSE_PARENTHESIS === $this->tokens[ $i ]['code']
+			) {
 				continue;
 			}
 
@@ -556,15 +560,6 @@ protected function check_code_is_escaped( $start, $end ) {
 				continue;
 			}
 
-			if ( \in_array( $this->tokens[ $i ]['code'], array( \T_DOUBLE_ARROW, \T_CLOSE_PARENTHESIS ), true ) ) {
-				continue;
-			}
-
-			// Handle magic constants for debug functions.
-			if ( isset( Tokens::$magicConstants[ $this->tokens[ $i ]['code'] ] ) ) {
-				continue;
-			}
-
 			// Handle safe PHP native constants.
 			if ( \T_STRING === $this->tokens[ $i ]['code']
 				&& isset( $this->safe_php_constants[ $this->tokens[ $i ]['content'] ] )