On Thu, 17 Nov 2022 15:51:50 GMT, Alexander Zvegintsev <[email protected]> wrote:
> This test needs more stabilization: > > - The test can be easily hang by replacing <br> iteration on > `keycharHash.elements()` with `keycharHash.forEach((k, v) -> > punchCtrlKey(robot, v));` > I do not see any reason why we need a synchronized `Hashtable`, so switched > to `HashMap` > Looks like it might be a reason of original test hanging. > - `keyRelease`, `dispose` calls wrapped with finally block > - Just to be safe added timeout `@run main/timeout=600` ( none of the CI test > runs took too long thought) > > CI testing looks good on all platforms. This pull request has now been integrated. Changeset: 6d6046b3 Author: Alexander Zvegintsev <[email protected]> URL: https://git.openjdk.org/jdk/commit/6d6046b3799217c281d077f12bce1ec590149849 Stats: 229 lines in 2 files changed: 56 ins; 55 del; 118 mod 8252713: jtreg time out of CtrlASCII.java seems to hang the Xserver. Reviewed-by: prr ------------- PR: https://git.openjdk.org/jdk/pull/11214
