From adeb7b2fe692c52dd7513409bea2e9b5420ccee3 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..b5f605a6 100644 --- a/lib/parse.js +++ b/lib/parse.js @@ -167,6 +167,14 @@ function is_identifier_char(ch) { ; }; +function is_identifier_string(string){ + for(var i in string) { + if(!is_identifier_char(string[i])) + return false; + } + return true; +} + function parse_js_number(num) { if (RE_HEX_NUMBER.test(num)) { return parseInt(num.substr(2), 16);