RFR: 8257810: Only First page are printed in JTable.scrollRectToVisible [v2]
Prasanta Sadhukhan
psadhukhan at openjdk.java.net
Thu Apr 7 13:35:27 UTC 2022
On Thu, 7 Apr 2022 11:48:19 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> Prasanta Sadhukhan has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Test updated
>> - Test updated
>
> test/jdk/javax/swing/JTable/PrintAllPagesTest.java line 53:
>
>> 51: PrintAllPages test = new PrintAllPages(latch);
>> 52: Thread T1 = new Thread(test);
>> 53: T1.start();
>
> I wonder why you need another thread to create UI. You can create UI from calling `invokeAndWait` from main thread as it's usually done and then start to wait on the latch.
>
> The two classes can be merged into one.
I followed the manual test I created for other printing related tests but anyways I have updated as per suggestions.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8141
More information about the client-libs-dev
mailing list