Fixed bug #636 : Some class names cause CSS tokenizer to hang

squizlabs · Jul 6, 2015 · 84c1532 · 84c1532
1 parent d2cc7e8
commit 84c1532
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/CodeSniffer/Tokenizers/CSS.php b/CodeSniffer/Tokenizers/CSS.php
@@ -78,6 +78,10 @@ public function tokenizeString($string, $eolChar='\n')
             if ($token['code'] === T_BREAK
                 || $token['code'] === T_LIST
                 || $token['code'] === T_DEFAULT
+                || $token['code'] === T_SWITCH
+                || $token['code'] === T_FOR
+                || $token['code'] === T_FOREACH
+                || $token['code'] === T_WHILE
             ) {
                 $token['type'] = 'T_STRING';
                 $token['code'] = T_STRING;

diff --git a/package.xml b/package.xml
@@ -34,6 +34,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
     -- Thanks to Stefan Lenselink for the contribution
   - Generic TodoSniff and FixmeSniff are now better at processing strings with invalid characters
   - Squiz ControlSignatureSniff now supports control structures with only inline HTML content
+  - Fixed bug #636 : Some class names cause CSS tokenizer to hang
  </notes>
  <contents>
   <dir name="/">