Squiz/NonExecutableCode: fix undefined index during live coding reviews

src/Standards/Squiz/Sniffs/PHP/NonExecutableCodeSniff.php

@@ -225,25 +225,34 @@ public function process(File $phpcsFile, $stackPtr)
         for ($i = ($start + 1); $i < $end; $i++) {
             if (isset(Tokens::$emptyTokens[$tokens[$i]['code']]) === true
                 || isset(Tokens::$bracketTokens[$tokens[$i]['code']]) === true
+                || $tokens[$i]['code'] === T_SEMICOLON
             ) {
                 continue;
             }
 
             // Skip whole functions and classes/interfaces because they are not
             // technically executed code, but rather declarations that may be used.
-            if ($tokens[$i]['code'] === T_FUNCTION
-                || $tokens[$i]['code'] === T_CLASS
-                || $tokens[$i]['code'] === T_INTERFACE
+            if (isset(Tokens::$ooScopeTokens[$tokens[$i]['code']]) === true
+                || $tokens[$i]['code'] === T_FUNCTION
+                || $tokens[$i]['code'] === T_CLOSURE
             ) {
+                if (isset($tokens[$i]['scope_closer']) === false) {
+                    // Parse error/Live coding.
+                    return;
+                }
+
                 $i = $tokens[$i]['scope_closer'];
                 continue;
             }
 
             $line = $tokens[$i]['line'];
             if ($line > $lastLine) {
                 $type    = substr($tokens[$stackPtr]['type'], 2);
-                $warning = 'Code after %s statement cannot be executed';
-                $data    = [$type];
+                $warning = 'Code after %s statement (line %s) cannot be executed';
+                $data    = [
+                    $type,
+                    $tokens[$stackPtr]['line'],
+                ];
                 $phpcsFile->addWarning($warning, $i, 'Unreachable', $data);
                 $lastLine = $line;
             }

src/Standards/Squiz/Tests/PHP/NonExecutableCodeUnitTest.inc

@@ -259,3 +259,22 @@ class HttpStatus
     const CONTINUE = 100;
     const SWITCHING_PROTOCOLS = 101;
 }
+
+interface ABC {
+	public function noError($name, $var);
+}
+
+trait Something {
+	function getReturnType() {
+	    echo 'no error';
+	}
+}
+
+$a = new class {
+    public function log($msg)
+    {
+        echo 'no error';
+    }
+};
+
+interface MyInterface {

src/Standards/Squiz/Tests/PHP/NonExecutableCodeUnitTest.php

@@ -69,6 +69,7 @@ public function getWarningList()
             234 => 1,
             235 => 2,
             239 => 1,
+            273 => 2,
         ];
 
     }//end getWarningList()