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

http://gwt-code-reviews.appspot.com/1443808/diff/1/dev/core/src/com/google/gwt/dev/jjs/impl/AutoboxUtils.java#newcode47
dev/core/src/com/google/gwt/dev/jjs/impl/AutoboxUtils.java:47: for
(JMethod method : wrapperType.getMethods()) {
On 2011/05/26 15:45:00, jbrosenberg wrote:
move initialization of boxSig & unboxSig before the start of the for
loop.

Done.  Whoops!!

http://gwt-code-reviews.appspot.com/1443808/diff/1/dev/core/src/com/google/gwt/dev/jjs/impl/AutoboxUtils.java#newcode97
dev/core/src/com/google/gwt/dev/jjs/impl/AutoboxUtils.java:97:
JMethodCall argMethodCall = (JMethodCall) arg;
Yeah, you cannot assign into a method call.  Code like "foo() = 3" is an
error, so is "foo()++".  The only time it should ever happen is exactly
this case that this pass fixes.  So it really should just be an
assertion.

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

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

Reply via email to