Committed in r8509
http://code.google.com/p/google-web-toolkit/source/detail?r=8509
http://gwt-code-reviews.appspot.com/730802/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Did I already mention how awesome it is that you fixed this?
awesome!
/kelly
On Tue, Aug 10, 2010 at 5:05 PM, fre...@google.com wrote:
Committed in r8509
http://code.google.com/p/google-web-toolkit/source/detail?r=8509
http://gwt-code-reviews.appspot.com/730802/show
--
If you
http://gwt-code-reviews.appspot.com/730802/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
LGTM, except I kind of preferred the red. :)
http://gwt-code-reviews.appspot.com/730802/diff/11001/12003
File dev/core/src/com/google/gwt/dev/shell/ModuleSpace.java (right):
http://gwt-code-reviews.appspot.com/730802/diff/11001/12003#newcode387
http://gwt-code-reviews.appspot.com/730802/diff/11001/12003
File dev/core/src/com/google/gwt/dev/shell/ModuleSpace.java (right):
http://gwt-code-reviews.appspot.com/730802/diff/11001/12003#newcode387
dev/core/src/com/google/gwt/dev/shell/ModuleSpace.java:387:
onModuleLoad() threw an unexpected
On Wed, Aug 4, 2010 at 2:42 PM, sco...@google.com wrote:
LGTM, except I kind of preferred the red. :)
Patches welcome :-)
http://gwt-code-reviews.appspot.com/730802/diff/11001/12003
File dev/core/src/com/google/gwt/dev/shell/ModuleSpace.java (right):
http://gwt-code-reviews.appspot.com/730802/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
LGTM
http://gwt-code-reviews.appspot.com/730802/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
LGTM, but we should prefix the method with __gwt_ for namespacing.
http://gwt-code-reviews.appspot.com/730802/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
http://gwt-code-reviews.appspot.com/730802/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
10 matches
Mail list logo