On Wed, 31 Jan 2024 17:27:15 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> src/java.desktop/share/classes/javax/swing/plaf/basic/BasicTreeUI.java line 
>> 3284:
>> 
>>> 3282:                 if(tree != null) {
>>> 3283:                     rendererPane.removeAll();
>>> 3284:                     // Only ever removed when UI changes, this is OK!
>> 
>> The comment no longer applies—you remove all the components from 
>> `rendererPane` on the line above.
>
> This is the only important thing that prevents me from approving this PR. The 
> comment is misleading now.

OK, modified

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17458#discussion_r1473741414

Reply via email to