Roberto Lublinerman has posted comments on this change.

Change subject: Adds part of Java 7 new language features.
......................................................................


Patch Set 8:

(7 comments)

@Ray, can you have a look at this patch and its successor and if ok +2 them.

....................................................
File dev/core/src/com/google/gwt/dev/javac/testing/impl/Java7MockResources.java Line 22: public static final MockJavaResource INTEGERLITERALS = new MockJavaResource(
Done


Line 26:       StringBuilder code = new StringBuilder();
I followed JavaResouceBase.java.  But I agree, will fix here.


....................................................
File dev/core/src/com/google/gwt/dev/util/arg/OptionSource.java
Line 26:   enum SourceLevel {
Done


Line 27:     _6("1.6", "6"),
Done


Line 39:      * Returns a string value representation for the source level.
Added an explanation in the class doc. Done


....................................................
File dev/core/test/com/google/gwt/dev/CompilerTest.java
Line 60: assertEquals(OptionSource.SourceLevel._7, options.getSourceLevel());
Done. Made SourceLevel a top level enum.


....................................................
File user/test-super/com/google/gwt/dev/jjs/super/com/google/gwt/dev/jjs/test/Java7Test.java Line 27: * IMPORTANT: For each test here there must exists the corresponding method in the non super sourced
Done


--
To view, visit https://gwt-review.googlesource.com/2650
To unsubscribe, visit https://gwt-review.googlesource.com/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I91c1f39ff20a2e7ac131d647bf4c96e34ce47a70
Gerrit-PatchSet: 8
Gerrit-Project: gwt
Gerrit-Branch: master
Gerrit-Owner: Roberto Lublinerman <[email protected]>
Gerrit-Reviewer: Brian Slesinsky <[email protected]>
Gerrit-Reviewer: John Stalcup <[email protected]>
Gerrit-Reviewer: Leeroy Jenkins <[email protected]>
Gerrit-Reviewer: Matthew Dempsky <[email protected]>
Gerrit-Reviewer: Matthew Dempsky <[email protected]>
Gerrit-Reviewer: Ray Cromwell <[email protected]>
Gerrit-Reviewer: Roberto Lublinerman <[email protected]>
Gerrit-Reviewer: Roberto Lublinerman <[email protected]>
Gerrit-Reviewer: Thomas Broyer <[email protected]>
Gerrit-HasComments: Yes

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
--- You received this message because you are subscribed to the Google Groups "GWT Contributors" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to