Thanks, I'll look at this today. On Mar 24, 2011 9:08 AM, "Akito Nozaki" <akito.noz...@gmail.com> wrote: > Not sure how this contributing thing works. I was assuming that things get
> pushed here automatically if I created an issue at the review site. > > I uploaded some code to deal with memory leak in ResettableEventBus. Mainly > with nested ResettableEventBus and passing ResettableEventBus into > ActivityManager. > > http://code.google.com/p/google-web-toolkit/issues/detail?id=5700 > http://gwt-code-reviews.appspot.com/1388804/ > > Really simple fix and would really love to get this included in the next > release as I keep having to patch/include my own ResettableEventBus in my > code. > > As I was working on this code I noticed that CountingEventBus has the same > bug but I'm guessing that only gets used in tests? Also I wasn't sure how to > write a test that check the internal state. Mainly to test for the memory > leak the current ResettableEventBus has. > > Akito > > -- > http://groups.google.com/group/Google-Web-Toolkit-Contributors -- http://groups.google.com/group/Google-Web-Toolkit-Contributors