On Mon, 31 Oct 2022 19:08:14 GMT, Chris Plummer <[email protected]> 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. This pull request has now been integrated. Changeset: 2f7dc5c4 Author: Chris Plummer <[email protected]> URL: https://git.openjdk.org/jdk/commit/2f7dc5c4cde32978c46d092d593c5c76df1edc18 Stats: 370 lines in 3 files changed: 7 ins; 348 del; 15 mod 8296089: Remove debug agent code for special handling of Thread.resume() Reviewed-by: alanb ------------- PR: https://git.openjdk.org/jdk/pull/10922
