Changeset 4008

Timestamp:

07/29/09 17:05:08 (4 years ago)

Author:

wwalc

Message:

#4140 - Releaser: fixed improper compression of Arrays

Location:

CKPackager/trunk

Files:

• _source/includes/scriptcompressor.js (modified) (1 diff)
• bin/ckpackager.exe (modified) (previous)
• bin/ckpackager.jar (modified) (previous)
• test/test.js (modified) (1 diff)

CKPackager/trunk/_source/includes/scriptcompressor.js

@@ -303 +303 @@
                                 if ( child.getType() == Token.NAME )
                                 {
-                                        if ( child.getString() == 'Array' && ( !child.getNext() || child.getNext().getType() != Token.NUMBER || child.getNext().getNext() ) )
+                                        // Simplify the Array declaration if it has no arguments, or at least two of them.
+                                        // new Array() => []
+                                        // new Array( 5, 10 ) => [5,10]
+                                        // new Array( 5 ) => new Array(5)
+                                        if ( child.getString() == 'Array' && ( !child.getNext() || child.getNext().getNext() ) )
                                         {
                                                 out( '[' );

CKPackager/trunk/test/test.js

@@ -138 +138 @@
                         "var a=[2,'fred'];" ],
 
-                [       "var a=new Array(b);",
-                        "var a=[b];" ],
+                [       "var a=new Array(b);" ],
+
+                [       "var a=new Array(b+1);" ],
 
                 [       "var a=new Object();",