On Thu, 7 Apr 2022 11:48:19 GMT, Alexey Ivanov <aiva...@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
