Squiz.PHP.EmbeddedPhp: prevent removing embedded PHP when no space between comment and close tag

When there is no space between a `/* .. */` comment and the close tag, PHPCS would report an `Empty embedded PHP tag found (Squiz.PHP.EmbeddedPhp.Empty)` error and the fixer would accidentally remove a complete line of embedded PHP. Includes unit tests.

Author: jrfnl

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

@@ -308,7 +308,7 @@ private function validateInlineEmbeddedPhp($phpcsFile, $stackPtr)
  }
 
  // Check that there is one, and only one space at the start of the statement.
- $firstContent = $phpcsFile->findNext(T_WHITESPACE, ($stackPtr + 1), ($closeTag - 1), true);
+ $firstContent = $phpcsFile->findNext(T_WHITESPACE, ($stackPtr + 1), $closeTag, true);
 
  if ($firstContent === false) {
  $error = 'Empty embedded PHP tag found';

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

@@ -113,3 +113,5 @@ function foo()
 <?php echo 'oops'; // Something.?>
 
 <?php /* translators: My sites label */ ?>
+<?php /* translators: My sites label */?>
+<?php /* translators: My sites label */ ?>

src/Standards/Squiz/Tests/PHP/EmbeddedPhpUnitTest.inc.fixed

@@ -113,3 +113,5 @@ function foo()
 <?php echo 'oops'; // Something. ?>
 
 <?php /* translators: My sites label */ ?>
+<?php /* translators: My sites label */ ?>
+<?php /* translators: My sites label */ ?>

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

@@ -53,6 +53,8 @@ public function getErrorList()
  102 => 1,
  112 => 1,
  113 => 1,
+ 116 => 1,
+ 117 => 1,
  );
 
  }//end getErrorList()