From 134b25a63d83a80ae128aedc9a99987e064b4382 Mon Sep 17 00:00:00 2001
From: Matthias Mullie <git@mullie.eu>
Date: Fri, 15 Sep 2017 15:02:08 +0200
Subject: [PATCH] Don't confuse comments with regex

Fixed #199
---
 src/JS.php          |  2 +-
 tests/js/JSTest.php | 18 ++++++++++++++++++
 2 files changed, 19 insertions(+), 1 deletion(-)

diff --git a/src/JS.php b/src/JS.php
index 042f542..80302af 100644
--- a/src/JS.php
+++ b/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
diff --git a/tests/js/JSTest.php b/tests/js/JSTest.php
index b9aa809..147e765 100644
--- a/tests/js/JSTest.php
+++ b/tests/js/JSTest.php
@@ -1132,6 +1132,24 @@ function someOtherFunction() {
             'if(!e.allow_html_data_urls&&V.test(k)&&!/^data:image\//i.test(k))return',
         );
 
+        // https://github.com/matthiasmullie/minify/issues/199
+        $tests[] = array(
+            '// This case was fixed on version 1.3.50
+// function () {
+//    return false;
+// };
+
+// Next two cases failed since version 1.3.49
+// function () {
+//    return false; //.click();
+// };
+
+// function () {
+//    ;//;
+// }',
+            '',
+        );
+
         // known minified files to help doublecheck changes in places not yet
         // anticipated in these tests
         $files = glob(__DIR__.'/sample/minified/*.js');