Thanks. done and commited as http://code.google.com/p/google-web-toolkit/source/detail?r=8868
On Fri, Sep 24, 2010 at 5:09 PM, <rj...@google.com> wrote: > LGTM > > one name nit > > > http://gwt-code-reviews.appspot.com/920801/diff/3/3001 > File > > > user/src/com/google/gwt/requestfactory/client/impl/DeltaValueStoreJsonImpl.java > (right): > > http://gwt-code-reviews.appspot.com/920801/diff/3/3001#newcode128 > > user/src/com/google/gwt/requestfactory/client/impl/DeltaValueStoreJsonImpl.java:128: > public void commit(JavaScriptObject returnedJso) { > Rename to something more evocative, e.g. processFuturesAndPostEvents (if > that's accurate) > > > http://gwt-code-reviews.appspot.com/920801/show > -- http://groups.google.com/group/Google-Web-Toolkit-Contributors