RFR: 8319192: Remove javax.swing.plaf.synth.SynthLookAndFeel.load(URL url) [v2]

Phil Race prr at openjdk.org
Wed Mar 5 23:07:07 UTC 2025


On Wed, 5 Mar 2025 22:47:11 GMT, Phil Race <prr at openjdk.org> wrote:

> > It is also mentioned in
> > https://github.com/openjdk/jdk/blob/caaf4098452476d981183ad4302b76b9c883a72b/src/java.desktop/share/classes/javax/swing/plaf/synth/doc-files/synthFileFormat.html#L57-L58
> 
> Ah. I'd forgotten that. I had updated it back in JDK 21 but it slipped my mind this time round.

fixed

>> 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.

fixed

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

PR Comment: https://git.openjdk.org/jdk/pull/23909#issuecomment-2702278386
PR Review Comment: https://git.openjdk.org/jdk/pull/23909#discussion_r1982310621


More information about the client-libs-dev mailing list