On Thu, 10 Nov 2022 20:58:36 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> The debug agent sets a breakpoint in Thread.resume() so it can prevent the >> debugger from suspending threads while in the resume call: >> >> /* >> * Track the resuming thread by marking it as being within >> * a resume and by setting up for notification on >> * a frame pop or exception. We won't allow the debugger >> * to suspend threads while any thread is within a >> * call to resume. This (along with the block below) >> * ensures that when the debugger >> * suspends a thread it will remain suspended. >> */ >> trackAppResume(resumer); >> >> Now that Thread.resume() is unsupported and just throws >> UnsupportedOperationException, all debug agent code related to this support >> can be removed. It's at least a couple of hundred lines of code, and with a >> fair amount of confusing synchronization. It will be nice to see it go. > > Chris Plummer has updated the pull request incrementally with one additional > commit since the last revision: > > Add some comments for newly added asserts Thanks for the reviews Serguei and Alan! ------------- PR: https://git.openjdk.org/jdk/pull/10922
