On Wed, 15 Sep 2021 18:29:44 GMT, Lance Andersen <[email protected]> wrote:
>>> Maybe this should use the same pattern than the other property above:
>>>
>>> ```
>>> ENABLE_JAR_INDEX = p != null ? p.equals("true") || p.isEmpty() : false;
>>> ```
>>
>> Yes, the intention is that -Djdk.net.URLClassLoader.enableJarIndex will
>> re-enable the feature.
>
> 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...
-------------
PR: https://git.openjdk.java.net/jdk/pull/5524