[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-serviceability-dev
Subject: Integrated: 8282076: Merge some debug agent changes from the loom repo
From: Chris Plummer <cjplummer () openjdk ! java ! net>
Date: 2022-02-23 21:12:55
Message-ID: JbwKTCLa8J7pfLqof5Hk12PTYS_SKzPsrpOloTuAWf8=.f52ff2cb-118e-4370-a1c0-d12c44d2bcd0 () github ! com
[Download RAW message or body]
On Thu, 17 Feb 2022 21:32:14 GMT, Chris Plummer <cjplummer@openjdk.org> wrote:
> There are some minor debug agent changes in the loom repo that are not virtual \
> thread specific, and I would like to merge them into the jdk repo. This will make \
> the future real merge of loom into jdk a bit cleaner.
> The biggest change is moving some code into the new filterAndHandleEvent() \
> function. However, there are no logic changes. It's the same code, just moved into \
> its own function.
> In handleMethodEnterEvent() I reworked some confusing logic and also the comment \
> that goes with it, but it should be functionally the same.
> ### Tests:
> - [x] vmTestbase/nsk/jdb
> - [x] vmTestbase/nsk/jdi
> - [x] vmTestbase/nsk/jdwp
> - [x] serviceability/jdwp
> - [x] jdk/com/sun/jdi
This pull request has now been integrated.
Changeset: 253cf785
Author: Chris Plummer <cjplummer@openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/253cf7852f60ecf92e6d675ae2469e5f27425609
Stats: 117 lines in 3 files changed: 61 ins; 45 del; 11 mod
8282076: Merge some debug agent changes from the loom repo
Reviewed-by: amenkov, lmesnik
-------------
PR: https://git.openjdk.java.net/jdk/pull/7524
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic