On Wed, 15 Sep 2021 18:59:08 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

>> I assume the reason for specifically checking for a value of true or an 
>> empty string as you envision allowing the property to be explicitly set to 
>> false in the future?  
>> 
>> Otherwise we could just check if the property is set(regardless of value) to 
>> re-enable the feature.
>
> I would dislike it if `-Djdk.net.URLClassPath.enableJarIndex=false` meant 
> true...

I can appreciate that and that is a fair point.   I guess  where I was coming 
from was that the property is temporary and has to be explicitly specified so 
ignoring the value seemed worth raising/discussing.   

 Assuming the current check is kept(which we should based on your input), it 
should probably be changed to ignore case.

-------------

PR: https://git.openjdk.java.net/jdk/pull/5524

Reply via email to