RFR: 8229867: Re-examine synchronization usages in http and https protocol handlers [v2]

Daniel Fuchs dfuchs at openjdk.java.net
Fri Oct 9 12:08:44 UTC 2020


On Fri, 9 Oct 2020 08:49:54 GMT, Chris Hegarty <chegar at openjdk.org> wrote:

>> Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8229867: Re-examine synchronization usages in http and https protocol handlers
>>   
>>   Incorporated review feedback
>
> src/java.base/share/classes/sun/net/www/http/KeepAliveStream.java line 69:
> 
>> 67:     public void close() throws IOException  {
>> 68:         // If the inputstream is closed already, or if this stream
>> 69:         // has already been queued for cleanup.just return.
> 
> Minor typo, "cleanup.just"

done

> src/java.base/share/classes/sun/net/www/MessageHeader.java line 330:
> 
>> 328:         at the end. Omits pairs with a null key. Omits
>> 329:         colon if key-value pair is the requestline. */
>> 330:     private  void print(int nkeys, String[] keys, String[] values, PrintStream p) {
> 
> While not strictly necessary, this method (along with isRequestline) could be _static_. Which ensures that their
> implementations do not access instance fields.

Done.

-------------

PR: https://git.openjdk.java.net/jdk/pull/558



More information about the security-dev mailing list