LGTM

http://gwt-code-reviews.appspot.com/1246801/diff/1/2
File user/src/com/google/gwt/i18n/client/Messages.java (right):

http://gwt-code-reviews.appspot.com/1246801/diff/1/2#newcode204
user/src/com/google/gwt/i18n/client/Messages.java:204: * </pre></code>
Is it safe to explicitly rely on annotation order in Java?

http://gwt-code-reviews.appspot.com/1246801/diff/1/6
File user/src/com/google/gwt/i18n/rebind/MessagesMethodCreator.java
(right):

http://gwt-code-reviews.appspot.com/1246801/diff/1/6#newcode1301
user/src/com/google/gwt/i18n/rebind/MessagesMethodCreator.java:1301:
writer.println("switch (arg" + listArgNum + "_size) {");
Missing writer.indent();

http://gwt-code-reviews.appspot.com/1246801/diff/1/6#newcode1352
user/src/com/google/gwt/i18n/rebind/MessagesMethodCreator.java:1352:
writer.outdent();
Missing writer.outdent();

http://gwt-code-reviews.appspot.com/1246801/show

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

Reply via email to