On Mon, 18 Apr 2022 14:33:35 GMT, Roger Riggs <rri...@openjdk.org> wrote:
> Please combine these closely related cases into a single test, preferably > with TestNG or JUnit. I normally prefer to have one job in one test, so that it is easier for debugging. I'm fine to combine them as the cases are simple enough. > test/jdk/javax/security/auth/callback/ClearPasswordMethod.java line 53: > >> 51: System.gc(); >> 52: Thread.sleep(100); >> 53: } > > The test could complete more quickly if the loop was exited when whm.size() > == 0. > And in the other test. Good point. It has been a concern to me to have the threads sleep. I will check other PRs, and add the size checking as well. ------------- PR: https://git.openjdk.java.net/jdk/pull/8272
