From 9af2bbffde2653237b36a06b005209be9f6cc1e6 Mon Sep 17 00:00:00 2001 From: Justin Lau Date: Mon, 6 May 2013 02:45:41 +0800 Subject: [PATCH] Fixed dot properties not optimizing unicode identifiers. Signed-off-by: Justin Lau --- lib/compress.js | 4 ++-- lib/parse.js | 8 ++++++++ 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/lib/compress.js b/lib/compress.js index 623fe45e..992d78f8 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -1962,8 +1962,8 @@ merge(Compressor.prototype, { var prop = self.property; if (prop instanceof AST_String && compressor.option("properties")) { prop = prop.getValue(); - if (is_identifier(prop) - || (compressor.option("screw_ie8") && /^[a-z_$][a-z0-9_$]*$/i.test(prop))) { + if (compressor.option("screw_ie8") && RESERVED_WORDS(prop) + || !(RESERVED_WORDS(prop)) && is_identifier_string(prop)) { return make_node(AST_Dot, self, { expression : self.expression, property : prop diff --git a/lib/parse.js b/lib/parse.js index da39b5b1..b3687201 100644 --- a/lib/parse.js +++ b/lib/parse.js @@ -167,6 +167,14 @@ function is_identifier_char(ch) { ; }; +function is_identifier_string(str){ + for (var i = str.length; --i >= 0;) { + if (!is_identifier_char(str.charAt(i))) + return false; + } + return true; +}; + function parse_js_number(num) { if (RE_HEX_NUMBER.test(num)) { return parseInt(num.substr(2), 16); -- 2.34.1