I didn't look thoroughly at the changes but I like the overall changes (wait 'til all invocations are processed to resolve properties + splitting property resolving and client peer construction), and as long as the tests are green… ;-)
http://gwt-code-reviews.appspot.com/1453813/diff/1/user/src/com/google/web/bindery/requestfactory/server/Resolver.java File user/src/com/google/web/bindery/requestfactory/server/Resolver.java (right): http://gwt-code-reviews.appspot.com/1453813/diff/1/user/src/com/google/web/bindery/requestfactory/server/Resolver.java#newcode191 user/src/com/google/web/bindery/requestfactory/server/Resolver.java:191: assert !(clientObject instanceof Resolution); Could it ever happen? http://gwt-code-reviews.appspot.com/1453813/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
