Added a fix I needed on top of this patch for an unrelated issue trying
to access ::class fields in jsni.  I know it should be in its own issue,
but until this patch goes through, it will just make merging confusing.


http://gwt-code-reviews.appspot.com/1801804/diff/1/core/src/com/google/gwt/dev/shell/DispatchClassInfo.java
File core/src/com/google/gwt/dev/shell/DispatchClassInfo.java (left):

http://gwt-code-reviews.appspot.com/1801804/diff/1/core/src/com/google/gwt/dev/shell/DispatchClassInfo.java#oldcode237
core/src/com/google/gwt/dev/shell/DispatchClassInfo.java:237: cls,
true);
Instead of sending true to findMostDerived, can we send cls !=
Class.class; if you try to use @com.foo.SomeClass::class in jsni, dev
mode will blow up (security exception trying to make Class constructor
accessible).

I would open a new code review for it, but until this is fixed in trunk,
there's little sense in fixing it until this patch goes through.

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

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

Reply via email to