On 2010/09/27 23:49:02, rjrjr wrote:
Point taken on the view contention front. I think Thomas is right that setting and clearing the view delegate should happen in start() and
stop(),
not in the constructor. Thanks, Thomas.
You're welcome ;-)
> > Sounds like the javadoc should say so. > > +1
How about also adding an assert in the start method? assert view != null : "Activity cannot be reused"; (in both AbstractProxyListActivity and AbstractProxyEditActivity) (I'd actually even add a isDead() protected method for the "view == null" check) http://gwt-code-reviews.appspot.com/925801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
