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