Ticket #4692: 4692_2.patch

_source/includes/scriptcompressor.js

@@ -76 +76 @@
         precedence[ Token.COMMA ] = 17;
 
         // Tokens having right-to-left associativity.
-        // From http://developer.mozilla.org/en/docs/Core_JavaScript_1.5_Reference:Operators:Operator_Precedence
+        // From https://developer.mozilla.org/En/Core_JavaScript_1.5_Reference/Operators/Operator_Precedence
         var associativityRTL = {};
         associativityRTL[ Token.SETNAME ] =
         associativityRTL[ Token.SETPROP ] =
@@ -429 +429 @@
                                 var param = child.getNext();
                                 while( param )
                                 {
+                                        if ( param.getType() == Token.COMMA )
+                                                out( '(' );
+
                                         if ( !writeNode( param ) )
                                                 break;
 
+                                        if ( param.getType() == Token.COMMA )
+                                                out( ')' );
+
                                         param = param.getNext();
 
                                         if ( param )
@@ -1103 +1109 @@
                                 out( lang[ type ] );
 
                                 // The right side.
-                                if ( !parenthesis )
-                                        parenthesis = ( precedence[ type ] < ( precedence[ right.getType() ] || 0 ) );
+                                parenthesis = ( precedence[ type ] < ( precedence[ right.getType() ] || 0 ) );
 
                                 if ( !parenthesis && !associativityRTL[ type ] && !noAssociativity[ type ] )
                                         parenthesis = ( precedence[ type ] == ( precedence[ right.getType() ] || 0 ) );

test/test.js

@@ -288 +288 @@
                 [       "var a=(b)?c:d;",
                         "var a=b?c:d;" ],
 
-                [       "var a=(b=c)?d:e;" ]
+                [       "var a=(b=c)?d:e;" ],
+
+                [       "a((1,2),3);" ],
+
+                [       "(1,2)+3;" ],
+
+                [       "3+(1,2);" ],
+
+                [       "(1,2);", "1,2;" ]
         ];
 
         var tests =