On Wed, 5 Mar 2025 22:38:59 GMT, Alexander Zvegintsev <azveg...@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 
> 223:
> 
>> 221:         }
>> 222: 
>> 223:         assert(classResourceBase != null);
> 
> It looks like this assert is no longer needed, because the 
> `classResourceBase` is not null at this point.

Fair enough. But I'm not sure it was ever needed. The condition in the existing 
code also ensured it would never fail.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23909#discussion_r1982309622

Reply via email to