On Thu, 24 Apr 2025 05:28:09 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
> nsk.share.jdi.Debugee.waitingEvent() takes a timeout argument. However, if > the call to EventQueue.remove() times out and returns null, waitingEvent() > just keep on retrying. The logic is incorrect for a null result. It should > immediately return null. The retry path is only meant for filtering out other > events, and it updates "timeLeft' before retrying to avoid endlessly > repeating the loop. > > Tested by running all of nsk/jdi locally on linux-x64-debug This pull request has now been integrated. Changeset: 486acc06 Author: Chris Plummer <cjplum...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/486acc06e0325d247a96df8f7fc88c9111c3315d Stats: 4 lines in 1 file changed: 0 ins; 1 del; 3 mod 8355453: nsk.share.jdi.Debugee.waitingEvent() does not timeout properly Reviewed-by: lmesnik, amenkov, sspitsyn ------------- PR: https://git.openjdk.org/jdk/pull/24839
