On Wed, 8 Jun 2022 10:12:45 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> Tejesh R has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Updated for multiple LookAndFeel
>
> test/jdk/javax/swing/JTableHeader/TableHeaderRendererTest.java line 52:
> 
>> 50:         for (UIManager.LookAndFeelInfo look : lookAndFeel) {
>> 51:             // initialize should not throw NullPointerException
>> 52:             LnFName = look.getName();
> 
> Why not declare it here? It does not need to be a field, moreover you don't 
> need it even as a local variable.

Yeah, had a different thought about that before, later didn't change it. Will 
update.

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

PR: https://git.openjdk.java.net/jdk/pull/8830

Reply via email to