On Wed, 15 Sep 2021 18:29:44 GMT, Lance Andersen <lan...@openjdk.org> 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