Detect if bootclasspath/classpath is set in module mode

Karen Kinnear Karen.Kinnear at Sun.COM
Fri Mar 19 10:43:50 PDT 2010


Mandy,

They both look good. Glad you caught the early null termination and  
thanks for adding
comments about temporary paths :-)

thanks,
Karen

On Mar 19, 2010, at 1:35 PM, Mandy Chung wrote:

> Alan,
>
> I updated the webrev that renames is_modular_mode to is_module_mode  
> in hotspot and also fixes an embarrassing bug of mine in src/share/ 
> bin/java.c line 793 that writes '\0' in a bad location.
>
> Webrev at:
>  http://cr.openjdk.java.net/~mchung/jigsaw/modular-mode-checks/jdk.01/
>  http://cr.openjdk.java.net/~mchung/jigsaw/modular-mode-checks/hotspot.01/
>
> Thanks
> Mandy
>
> Mandy Chung wrote:
>> Karen, Alan,
>>
>> We don't support bootclasspath and classpath  in module mode.  I  
>> added the check in hotspot and java launcher.  Can you review this  
>> simple fix?
>>
>> Webrev at:
>>  http://cr.openjdk.java.net/~mchung/jigsaw/modular-mode-checks/
>>
>> Thanks
>> Mandy
>




More information about the jigsaw-dev mailing list