Merge pull request #6 from PHPCSStandards/feature/minor-regex-tweak

afilina · web-flow · commit 806997b53301 · 2021-03-17T13:03:54.000-04:00
SniffParser: minor regex simplification
diff --git a/src/Parser/SniffParser.php b/src/Parser/SniffParser.php
@@ -78,8 +78,8 @@ private function getSniffClassName(string $phpFilePath): string
      */
     private function getSniffFileParts(string $filePath): array
     {
-        $part = '([^\/]*)';
-        preg_match("/$part\/Sniffs\/$part\/{$part}Sniff\.php/", $filePath, $matches);
+        $part = '([^/]*)';
+        preg_match("`$part/Sniffs/$part/{$part}Sniff\.php$`", $filePath, $matches);
         if ($matches === []) {
             throw NotASniffPath::fromPath($filePath);
         }

Original file line number	Diff line number	Diff line change
`@@ -78,8 +78,8 @@ private function getSniffClassName(string $phpFilePath): string`
`78`	`78`	`*/`
`79`	`79`	`private function getSniffFileParts(string $filePath): array`
`80`	`80`	`{`
`81`		`- $part = '([^\/]*)';`
`82`		`- preg_match("/$part\/Sniffs\/$part\/{$part}Sniff\.php/", $filePath, $matches);`
	`81`	`+ $part = '([^/]*)';`
	`82`	+ preg_match("`$part/Sniffs/$part/{$part}Sniff\.php$`", $filePath, $matches);
`83`	`83`	`if ($matches === []) {`
`84`	`84`	`throw NotASniffPath::fromPath($filePath);`
`85`	`85`	`}`