Review request 8154837: Class::getPackage with exploded modules when classes in modules defined to the boot loader
Mandy Chung
mandy.chung at oracle.com
Fri Apr 22 14:45:55 UTC 2016
> On Apr 21, 2016, at 11:31 PM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
>
> On 21/04/2016 23:03, Mandy Chung wrote:
>> Webrev:
>> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8154837/webrev.00/
>>
>> The module location from an exploded image is file URL rather than file path. This issue can be reproduced with jdk/test/java/lang/Package/GetPackages.java on windows.
> This looks okay, maybe slightly better if it starts for "file:/“.
Will patch before the push.
> At some point then I think we need to re-examine the underlying JVM function to see if we can get it to return a file URL for -Xbootclasspath/a case too. That is, it's a bit strange for the function to return a String that is sometimes a URL, sometimes a file path.
I agree and it’d be better to be consistent. I will file an issue to track this.
Mandy
More information about the core-libs-dev
mailing list