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