RFR of JDK-8158149: test bug for SystemModuleFinder when fast path is supported
Mandy Chung
mandy.chung at oracle.com
Tue May 31 01:11:59 UTC 2016
>> On 30/05/2016 12:48, Hamlin Li wrote:
>>> Would you mind to review the following patch?
>>>
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8158149
>>>
>>> webrev: http://cr.openjdk.java.net/~mli/8158149/webrev.00/
>>>
>>>
>>> The system property to disable fast path should be "jdk.system.module.finder.disabledFastPath" rather than "jdk.installed.modules.disable”.
The fix looks fine. I missed this test when the system property was renamed.
We also renamed “InstalledModuleDescriptorPlugin" to “SystemModuleDescriptorPlugin” before we integrated jigsaw to jdk9. Do you have cycle to rename the directory “InstalledModuleDescriptors” and any reference to “installed” to “system” in the tests in a separate patch (a separate JBS issue)?
thanks
Mandy
More information about the jigsaw-dev
mailing list