LGTM with nits.
http://gwt-code-reviews.appspot.com/34836/diff/1001/2002
File dev/core/src/com/google/gwt/dev/shell/rewrite/HasAnnotation.java
(right):
http://gwt-code-reviews.appspot.com/34836/diff/1001/2002#newcode29
Line 29: * a type.
Should mention the annotation has to be directly on the type without
considering inheritance (perhaps on the hasAnnotation method instead of
here).
http://gwt-code-reviews.appspot.com/34836/diff/1001/2002#newcode50
Line 50: boolean found;
private?
http://gwt-code-reviews.appspot.com/34836/diff/1001/2003
File dev/core/super/com/google/gwt/core/client/BridgeClass.java (right):
http://gwt-code-reviews.appspot.com/34836/diff/1001/2003#newcode26
Line 26: * super-source.
Still not sold on the name, but can't think of anything better. If this
is intended to be something that external code might use, there should
probably be more documentation and an example. If not, it should
probably be documented as such.
http://gwt-code-reviews.appspot.com/34836/diff/1001/2005
File
user/test-super/com/google/gwt/dev/jjs/super/com/google/gwt/dev/jjs/bridge/UserBridgeClass.java
(right):
http://gwt-code-reviews.appspot.com/34836/diff/1001/2005#newcode32
Line 32: * Test cross-bounday method invocation.
typo
http://gwt-code-reviews.appspot.com/34836/diff/1001/2009
File user/test/com/google/gwt/dev/jjs/bridge/UserBridgeClass.java
(right):
http://gwt-code-reviews.appspot.com/34836/diff/1001/2009#newcode33
Line 33: * Test cross-bounday method invocation.
typo: bounday
http://gwt-code-reviews.appspot.com/34836
--~--~-~--~~~---~--~~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~--~~~~--~~--~--~---