Re: [gwt-contrib] Re: Fixes two Roo issues: ROO-1238 (Amit) and ROO-1380 (pdr) (issue897801)

Thanks Phil and Ray. On Sat, Sep 18, 2010 at 12:12 PM, Ray Ryan wrote: > LGTM++ > > > On Sat, Sep 18, 2010 at 12:11 PM, wrote: > >> http://gwt-code-reviews.appspot.com/897801/show >> > > -- > http://groups.google.com/group/Google-Web-Toolkit-Contributors > -- http://groups.google.com/group/G

[gwt-contrib] Re: Fixes two Roo issues: ROO-1238 (Amit) and ROO-1380 (pdr) (issue897801)

LGTM++ On Sat, Sep 18, 2010 at 12:11 PM, wrote: > http://gwt-code-reviews.appspot.com/897801/show > -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Fixes two Roo issues: ROO-1238 (Amit) and ROO-1380 (pdr) (issue897801)

http://gwt-code-reviews.appspot.com/897801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Fixes two Roo issues: ROO-1238 (Amit) and ROO-1380 (pdr) (issue897801)

LGTM http://gwt-code-reviews.appspot.com/897801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Fixes two Roo issues: ROO-1238 (Amit) and ROO-1380 (pdr) (issue897801)

http://gwt-code-reviews.appspot.com/897801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Fixes two Roo issues: ROO-1238 (Amit) and ROO-1380 (pdr) (issue897801)

LGTM with following addressed http://gwt-code-reviews.appspot.com/897801/diff/1/6 File samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseDetails.java (right): http://gwt-code-reviews.appspot.com/897801/diff/1/6#newcode952 samples/expenses/src/main/java/com/google/gwt/s