RFR 8172514: Stop checking for duplicate packages in the JVM_DefineModule() package list

[harold seigel]

Thanks Coleen!

Harold


On 3/9/2017 4:51 PM, coleen.phillimore at oracle.com wrote:
>
> This looks good.
> Coleen
>
> On 3/9/17 11:21 AM, harold seigel wrote:
>> Thanks Claes!
>>
>> Here's the correct open webrev URL: 
>> http://cr.openjdk.java.net/~hseigel/bug_8172514/webrev/index.html
>>
>> Harold
>>
>>
>> On 3/9/2017 11:15 AM, Claes Redestad wrote:
>>> Looks good to me!
>>>
>>> (You might want to host the webrev on cr.openjdk.java.net, though..)
>>>
>>> /Claes
>>>
>>> On 03/09/2017 04:44 PM, harold seigel wrote:
>>>> Hi,
>>>>
>>>> Please review this JDK-10 fix to removing checking for duplicate 
>>>> packages in the package list passed to JVM_DefineModule().  The 
>>>> checking is potentially expensive and also redundant because the 
>>>> JDK makes the same check.
>>>>
>>>>
>>>> JBS Bug: https://bugs.openjdk.java.net/browse/JDK-8172514
>>>>
>>>> The fix was tested with the JCK lang and vm tests, the JTreg 
>>>> hotspot, java/io, java/lang, java/util and other tests, the RBT 
>>>> tier2 -tier5 tests, the co-located and non-colocated NSK tests, and 
>>>> with JPRT.
>>>>
>>>> Thanks, Harold
>>>>
>>>
>>
>