Handle all types of single byte whitespace

src/JSMin/JSMin.php

@@ -123,16 +123,16 @@ public function min()
         while ($this->a !== null) {
             // determine next command
             $command = self::ACTION_KEEP_A; // default
-            if ($this->a === ' ') {
+            if ($this->isWhiteSpace($this->a)) {
                 if (($this->lastByteOut === '+' || $this->lastByteOut === '-')
                         && ($this->b === $this->lastByteOut)) {
                     // Don't delete this space. If we do, the addition/subtraction
                     // could be parsed as a post-increment
                 } elseif (! $this->isAlphaNum($this->b)) {
                     $command = self::ACTION_DELETE_A;
                 }
-            } elseif ($this->a === "\n") {
-                if ($this->b === ' ') {
+            } elseif ($this->isLineTerminator($this->a)) {
+                if ($this->isWhiteSpace($this->b)) {
                     $command = self::ACTION_DELETE_A_B;
 
                     // in case of mbstring.func_overload & 2, must check for null b,
@@ -143,8 +143,8 @@ public function min()
                     $command = self::ACTION_DELETE_A;
                 }
             } elseif (! $this->isAlphaNum($this->a)) {
-                if ($this->b === ' '
-                    || ($this->b === "\n"
+                if ($this->isWhiteSpace($this->b)
+                    || ($this->isLineTerminator($this->b)
                         && (false === strpos('}])+-"\'', $this->a)))) {
                     $command = self::ACTION_DELETE_A_B;
                 }
@@ -349,7 +349,7 @@ protected function get()
      */
     protected function isEOF($a)
     {
-        return ord($a) <= self::ORD_LF;
+        return $a === null || $this->isLineTerminator($a);
     }
 
     /**
@@ -451,4 +451,14 @@ protected function next()
         }
         return $get;
     }
+
+    protected function isWhiteSpace($s) {
+        // https://www.ecma-international.org/ecma-262/#sec-white-space
+        return strpos(" \t\v\f\xa0", $s) !== false;
+    }
+
+    protected function isLineTerminator($s) {
+        // https://www.ecma-international.org/ecma-262/#sec-line-terminators
+        return strpos("\n\r", $s) !== false;
+    }
 }

tests/JSMinTest.php

@@ -47,7 +47,7 @@ public function testMinify($testName, $input, $expected, $actualFile)
 	}
 
 	public function testWhitespace() {
-		$this->assertEquals("hello;", JSMin::minify("\r\n\r\nhello;\r\n"));
+		$this->assertEquals("hello;", JSMin::minify("\r\n\r\n \t\v\f\xa0hello;\r\n"));
 	}
 
 	public function testBomRemoval() {