hugoloza / gwt-platform

Automatically exported from code.google.com/p/gwt-platform
0 stars 0 forks source link

Manual bound checking for getCurrentPlaceRequest in PlaceManagerImpl #364

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Follow-up on the post at 
http://groups.google.com/group/gwt-platform/browse_thread/thread/8bf21467e9f2ec0

After I cross-posted to the GWT list, Thomas Broyer recommended checking for 
good values beforehand rather than relying on the exception handling: 
http://groups.google.com/group/google-web-toolkit/browse_thread/thread/ce87f7024
bd4ad62 

I've uploaded the patch to Rietveld: http://codereview.appspot.com/5050051

Thanks!
Tom

Original issue reported on code.google.com by TTerr...@gmail.com on 19 Sep 2011 at 7:05

GoogleCodeExporter commented 9 years ago

Original comment by philippe.beaudoin on 19 Sep 2011 at 7:08

GoogleCodeExporter commented 9 years ago
Thanks a lot for the patch, this has just landed! Please keep on contributing.

Original comment by philippe.beaudoin on 11 Oct 2011 at 4:54