On Fri, 14 Jun 2024 20:22:42 GMT, Phil Race <p...@openjdk.org> wrote:
>> Abhishek Kumar has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> condition update
>
> src/java.desktop/share/classes/javax/swing/plaf/synth/SynthGraphicsUtils.java
> line 785:
>
>> 783: }
>> 784:
>> 785: if (c instanceof Container) {
>
> JComponent extends Container .. so this will traverse everything in the
> Swing UI.
> Is there any pattern elsewhere in Swing you can use to short-circuit this ?
since this method should be recursing through all of a component's
subcomponents, shouldn't there not even be an if check here?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18992#discussion_r1640558484
