Code review: 7110484

Chris Hegarty chris.hegarty at oracle.com
Thu Nov 10 06:43:55 PST 2011


On 10/11/2011 14:27, Michael McMahon wrote:
> On 10/11/11 13:31, Michael McMahon wrote:
>> Chris,
>>
>> Would you mind taking a look at this? It's a small change to the http
>> server.
>> http://cr.openjdk.java.net/~michaelm/7110484/webrev.1/
>>
>> I haven't included a regression test because to test it reliably would
>> require
>> at least a minute's worth of test time. I'm not sure the issue
>> warrants that given
>> the recent improvements in test run time.
>>
>> Thanks
>> Michael
> Slightly improved version.
>
> http://cr.openjdk.java.net/~michaelm/7110484/webrev.2/

Looks good to me.

-Chris.



More information about the net-dev mailing list