RFR (XS) 8065096: java.net.Authenticator.theAuthenticator should be properly synchronized

Aleksey Shipilev aleksey.shipilev at oracle.com
Mon Nov 17 12:31:44 UTC 2014


Thanks for reviews!

Please push:
 http://cr.openjdk.java.net/~shade/8065096/8065096.changeset

-Aleksey.

On 11/17/2014 03:15 PM, Lance @ Oracle wrote:
> Looks fine aleksey
> 
> 
> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen|
> Principal Member of Technical Staff | +1.781.442.2037 <tel:+1.781.442.2037>
> Oracle Java Engineering 
> 1 Network Drive <x-apple-data-detectors://34/0>
> Burlington, MA 01803 <x-apple-data-detectors://34/0>
> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
> Sent from my iPad
> 
> On Nov 17, 2014, at 5:35 AM, Aleksey Shipilev
> <aleksey.shipilev at oracle.com <mailto:aleksey.shipilev at oracle.com>> wrote:
> 
>> Hi,
>>
>> Please review and commit this trivial change:
>> http://cr.openjdk.java.net/~shade/8065096/webrev.01/
>> <http://cr.openjdk.java.net/%7Eshade/8065096/webrev.01/>
>> https://bugs.openjdk.java.net/browse/JDK-8065096
>>
>> "volatile" should nominally fix the the data race on theAuthenticator
>> field.
>>
>> Thanks,
>> -Aleksey.
>>
>>





More information about the core-libs-dev mailing list