RFR: 8060130: Simplify the synchronization of defining and getting java.lang.Package

Peter Levart peter.levart at gmail.com
Mon Oct 13 15:52:21 UTC 2014


On 10/13/2014 05:41 PM, Claes Redestad wrote:
> On 10/13/2014 05:29 PM, Peter Levart wrote:
>> On 10/13/2014 05:15 PM, Peter Levart wrote:
>>> Yes, I realized I got this messed up and have prepared a new patch 
>>> which passes tests and includes your previous
>>> comments:
>>>
>>> http://cr.openjdk.java.net/~redestad/8060130/webrev.04 
>>
>>
>> Hi Claes,
>>
>> Hm, CachedManifest.getManifest() in 4th webrev is still a little 
>> faulty. It sometimes returns EMPTY_MANIFEST and sometimes null. I 
>> think it should always return null in case when no manifest is found 
>> (at least that was the old code behaviour).
>
> Hmm, yes, that last little detail of making sure we also set m before 
> returning:
>
> -                manifest = (m == null ? EMPTY_MANIFEST : m);
> +                manifest = m = (m == null ? EMPTY_MANIFEST : m);
>
> http://cr.openjdk.java.net/~redestad/8060130/webrev.05
>
> We rely on returning a non-null manifest from 
> CachedManifest.getManifest() in the current code, so returning null 
> would be a bad idea unless we revert the simplifications to 
> defineSystemPackage.

Right, I missed that part.

Peter

>
> /Claes
>
>>
>> Regards, Peter
>




More information about the core-libs-dev mailing list