RFR: 6257207: JTable.getDefaultEditor throws NullPointerException [v3]

[Prasanta Sadhukhan]

On Tue, 1 Nov 2022 16:45:45 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove extra line
>
> test/jdk/javax/swing/JTable/JTableEditorNPE.java line 62:
> 
>> 60:             });
>> 61:         } finally {
>> 62:             Thread.sleep(1000);
> 
> Is the `sleep` required?
> 
> If NPE is thrown, no UI appears at all; if NPE is not thrown, the UI may be shown but not required.
> 
> Would it be enough to add the table to the frame and call `validate()`?

I added sleep to see the frame before it dispose but I agree it's not required for checking the fix..Modified as per comment...

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

PR: https://git.openjdk.org/jdk/pull/10871