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/diff/11001/12003#newcode387
> dev/core/src/com/google/gwt/dev/shell/ModuleSpace.java:387:
> "onModuleLoad() threw an unexpected exception",
> I wouldn't necessarily call this "unexpected".. it's the user's
> exception so maybe it's unexpected, maybe it's not.
>
>
> http://gwt-code-reviews.appspot.com/730802/show
>



-- 
Fred Sauer
Developer Advocate
Google Inc.
1600 Amphitheatre Parkway
Mountain View, CA 94043
fre...@google.com

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

Reply via email to