On Wed, 5 Mar 2025 08:30:18 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org>
wrote:
>> Phil Race has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8319192
>
> src/java.desktop/share/classes/javax/swing/plaf/synth/SynthParser.java line
> 249:
>
>> 247: */
>> 248: private URL getResource(String path) {
>> 249: return _classResourceBase.getResource(path);
>
> Any particular reason for removing the null check?
Yes. the resource was *either* url-based or classpath based. One would be null,
the other not.
Now there's only one, so it must be the non-null one.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23909#discussion_r1982094177
