Require that a regex has some content

Otherwise, the `//` that start a comment could be misread
as regexes.

Fixes #196

Author: matthiasmullie

src/JS.php

@@ -242,7 +242,7 @@ protected function extractRegex()
                 (isset($match['after']) ? $match['after'] : '');
         };
 
-        $pattern = '(?P<regex>\/.*?(?<!\\\\)(\\\\\\\\)*\/[gimy]*)(?![0-9a-zA-Z\/])';
+        $pattern = '(?P<regex>\/(.|\\\\)+?((?<!\\\\)\\\\\\\\)*\/[gimy]*)(?![0-9a-zA-Z\/])';
 
         // a regular expression can only be followed by a few operators or some
         // of the RegExp methods (a `\` followed by a variable or value is

tests/js/JSTest.php

@@ -1115,6 +1115,17 @@ function someOtherFunction() {
             'if(!0||/^(https?:)?\/\//.test(\'xxx\'))alert(1)',
         );
 
+        // https://github.com/matthiasmullie/minify/issues/196
+        $tests[] = array(
+            'if ( true ) {
+    console.log(true);
+// ...comment number 2 (something with dots?)
+} else {
+    console.log(false);
+}',
+            'if(!0){console.log(!0)}else{console.log(!1)}',
+        );
+
         // known minified files to help doublecheck changes in places not yet
         // anticipated in these tests
         $files = glob(__DIR__.'/sample/minified/*.js');