On Thu, 16 Sep 2021 09:15:34 GMT, Daniel Fuchs <[email protected]> wrote:
>> 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.
>
> Given that there is a precedence in this file:
>
>
> p =
> props.getProperty("jdk.net.URLClassPath.showIgnoredClassPathEntries");
> DEBUG_CP_URL_CHECK = p != null ? p.equals("true") || p.isEmpty() :
> false;
>
>
> I would tend to let the new property follow the same pattern, especially
> since it's temporary.
Yes,I changed the code to follow the same pattern.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5524