Thomas Broyer has posted comments on this change.

Change subject: Allow line breaks and other whitespace in jsni method references.
......................................................................


Patch Set 1: Code-Review+2

(1 comment)

....................................................
File dev/core/test/com/google/gwt/dev/js/TokenStreamTest.java
Line 78:     assertBadJsni("@org.group.Foo::bar(\nLorg/group/Foo;,\nZ)");
Should we add something with a line-break in the middle of a L? e.g.

 assertBadJsni("@org.group.Foo::bar(Lorg/group/\nFoo;)");


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iecacf3b8be1e512a5fdf3132245091b4ebea9c68
Gerrit-PatchSet: 1
Gerrit-Project: gwt
Gerrit-Branch: master
Gerrit-Owner: Roberto Lublinerman <rlu...@google.com>
Gerrit-Reviewer: Daniel Kurka <danku...@google.com>
Gerrit-Reviewer: Leeroy Jenkins <jenk...@gwtproject.org>
Gerrit-Reviewer: Thomas Broyer <t.bro...@gmail.com>
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 google-web-toolkit-contributors+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to