Bob, this patch looks good.  But I have to admit I can't understand why
it's currently broken.

Explain to me again why this transformation breaks?
(a, b).foo() --> foo((a,b))

I didn't completely follow the pattern you showed.


http://gwt-code-reviews.appspot.com/78818/diff/1/2
File dev/core/src/com/google/gwt/dev/jjs/impl/MakeCallsStatic.java
(right):

http://gwt-code-reviews.appspot.com/78818/diff/1/2#newcode336
Line 336: if (x.getInstance() instanceof JMultiExpression) {
Nitpick: can you swap these two lines?

http://gwt-code-reviews.appspot.com/78818

--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---

Reply via email to