On Thu, 15 Aug 2024 21:12:20 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
> There are issues with the test attaching to the debuggee too soon, and the > debug agent isn't ready. yet. This test is based on ProcessAttachTest, which > does not have this issue. I eventually realized the reason why is because > ProcessAttachTest has this little snippet of code, which I had removed from > DataDumpTest: > > // Wait for the process to start > InputStream is = p.getInputStream(); > is.read(); > > This is waiting for the start of the debug agent's "Listening..." message. > I've re-added this to DataDumpTest, and it seems to fix the issue. Tested by > running 20 times with `-Xcomp` on all supported platforms with no failures. > It used to fail about 2/3 of the time. Thumbs up. Thanks for including `firstChar` in the debuggee output. I think this is a trivial fix. test/jdk/com/sun/jdi/DataDumpTest.java line 100: > 98: out.waitFor(); // Wait for the debuggee to exit > 99: > 100: System.out.println("Deuggee output:"); nit typo: s/Deuggee/Debuggee/ Not your typo, but could you fix it while you are there? ------------- Marked as reviewed by dcubed (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/20604#pullrequestreview-2243069811 PR Review Comment: https://git.openjdk.org/jdk/pull/20604#discussion_r1720054663