[gwt-contrib] Re: HandlerManager throws NPE if last handler is removed twice

2009-04-21 Thread rjrjr
http://gwt-code-reviews.appspot.com/25801/diff/1/3 File user/src/com/google/gwt/event/shared/HandlerManager.java (right): http://gwt-code-reviews.appspot.com/25801/diff/1/3#newcode101 Line 101: } Could be a little simpler--you don't really need an else case: boolean result = false; if (l !=

[gwt-contrib] Re: HandlerManager throws NPE if last handler is removed twice

2009-04-21 Thread jlabanca
See http://gwt-code-reviews.appspot.com/25803 for updated patch. On 2009/04/21 16:49:12, rjrjr wrote: http://gwt-code-reviews.appspot.com/25801/diff/1/3 File user/src/com/google/gwt/event/shared/HandlerManager.java (right): http://gwt-code-reviews.appspot.com/25801/diff/1/3#newcode101 Line