Request for Review: 6893238

[Christopher Hegarty - Sun Microsystems Ireland]

On 21/10/2009 15:47, Alan Bateman wrote:
> Christopher Hegarty - Sun Microsystems Ireland wrote:
>> :
>> Yes, this would certainly be cleaner. I've updated the webrev. Please 
>> take a look.
>>
>>   http://cr.openjdk.java.net/~chegar/6893238/webrev.1/webrev/
>>
>> Note: The lack of the initial cause of the IOException in firstToken 
>> should not be a problem since the Exception is swallowed in setHeaders 
>> ( a few lines above).
> This looks much better - thanks for doing this. Minor bit is that the 
> L183 can simply be "throw new IOException(...)".

Done.

Thanks,
-Chris.

> 
> -Alan.