Re: Review for fix to Bug 7078386

2011-12-16 Thread Chris Hegarty
On 12/16/11 10:40 AM, Alan Bateman wrote: On 15/12/2011 23:14, Brandon Passanisi wrote: Hello net-dev. I was wondering if somebody could review the proposed fix for the following bug: Bug URL: http://monaco.sfbay.sun.com/detail.jsf?cr=7078386 Webrev : http://cr.openjdk.java.net/~mich

Re: Review for fix to Bug 7078386

2011-12-16 Thread Alan Bateman
On 15/12/2011 23:14, Brandon Passanisi wrote: Hello net-dev. I was wondering if somebody could review the proposed fix for the following bug: Bug URL: http://monaco.sfbay.sun.com/detail.jsf?cr=7078386 Webrev : http://cr.openjdk.java.net/~michaelm/7078386/webrev.1/ The change looks g

Re: Review for fix to Bug 7078386

2011-12-16 Thread Michael McMahon
Brandon, The fix looks good to me. Thanks for contributing it. - Michael. On 15/12/11 23:14, Brandon Passanisi wrote: Hello net-dev.  I was wondering if somebody could review the proposed fix for the following bug: Bug URL: http:/