Oops, missed some of your questions. Applying the fixes and submitting.
http://gwt-code-reviews.appspot.com/755801/diff/5001/6002 File samples/dynatablerf/src/com/google/gwt/sample/dynatablerf/client/CalendarProvider.java (right): http://gwt-code-reviews.appspot.com/755801/diff/5001/6002#newcode50 samples/dynatablerf/src/com/google/gwt/sample/dynatablerf/client/CalendarProvider.java:50: On 2010/08/12 13:48:12, bobv wrote:
Extra whitespace. I remember having to mess with my checkstyle
settings to
convince eclipse it wanted to check stuff in the dynatablerf sample.
Done. http://gwt-code-reviews.appspot.com/755801/diff/5001/6002#newcode78 samples/dynatablerf/src/com/google/gwt/sample/dynatablerf/client/CalendarProvider.java:78: if (response.size() > 0) { Yes, if I understood your question. This is just a hack, an arbitrary trigger to find an arbitrary object to mess with to show you how it's done. I can't imagine actually doing the actual if check, I only put it there in case you managed to scroll off the end of the table. And it fails at that, because I'm messing with 1, not zero. Duh. Fixing. http://gwt-code-reviews.appspot.com/755801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
