From lkuskov at openjdk.org Mon Jun 9 21:33:14 2025 From: lkuskov at openjdk.org (Leonid Kuskov) Date: Mon, 9 Jun 2025 21:33:14 GMT Subject: RFR: 7904023: SHOULD NOT OCCUR: unknown cp_type Message-ID: [CODETOOLS-7904023](https://bugs.openjdk.org/browse/CODETOOLS-7904023) SHOULD NOT OCCUR: unknown cp_type ------------- Commit messages: - CODETOOLS-7904023: SHOULD NOT OCCUR: unknown cp_type Changes: https://git.openjdk.org/jcov/pull/59/files Webrev: https://webrevs.openjdk.org/?repo=jcov&pr=59&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7904023 Stats: 10 lines in 1 file changed: 2 ins; 2 del; 6 mod Patch: https://git.openjdk.org/jcov/pull/59.diff Fetch: git fetch https://git.openjdk.org/jcov.git pull/59/head:pull/59 PR: https://git.openjdk.org/jcov/pull/59 From shurailine at openjdk.org Tue Jun 10 07:43:57 2025 From: shurailine at openjdk.org (Alexandre Iline) Date: Tue, 10 Jun 2025 07:43:57 GMT Subject: RFR: 7904023: SHOULD NOT OCCUR: unknown cp_type In-Reply-To: References: Message-ID: On Mon, 9 Jun 2025 21:26:24 GMT, Leonid Kuskov wrote: > [CODETOOLS-7904023](https://bugs.openjdk.org/browse/CODETOOLS-7904023) SHOULD NOT OCCUR: unknown cp_type src/classes/com/sun/tdk/jcov/instrument/asm/ClassMorph.java line 417: > 415: i += 2; > 416: break; > 417: case 17: // dynamic So this is the real fix, everything else is cosmetic, am I reading it correctly? ------------- PR Review Comment: https://git.openjdk.org/jcov/pull/59#discussion_r2137145360 From shurailine at openjdk.org Tue Jun 10 08:22:41 2025 From: shurailine at openjdk.org (Alexandre Iline) Date: Tue, 10 Jun 2025 08:22:41 GMT Subject: RFR: 7904023: SHOULD NOT OCCUR: unknown cp_type In-Reply-To: References: Message-ID: On Mon, 9 Jun 2025 21:26:24 GMT, Leonid Kuskov wrote: > [CODETOOLS-7904023](https://bugs.openjdk.org/browse/CODETOOLS-7904023) SHOULD NOT OCCUR: unknown cp_type LGTM ------------- PR Comment: https://git.openjdk.org/jcov/pull/59#issuecomment-2958132189 From lkuskov at openjdk.org Tue Jun 10 17:25:49 2025 From: lkuskov at openjdk.org (Leonid Kuskov) Date: Tue, 10 Jun 2025 17:25:49 GMT Subject: Integrated: 7904023: SHOULD NOT OCCUR: unknown cp_type In-Reply-To: References: Message-ID: On Mon, 9 Jun 2025 21:26:24 GMT, Leonid Kuskov wrote: > [CODETOOLS-7904023](https://bugs.openjdk.org/browse/CODETOOLS-7904023) SHOULD NOT OCCUR: unknown cp_type This pull request has now been integrated. Changeset: e2c20f0a Author: Leonid Kuskov URL: https://git.openjdk.org/jcov/commit/e2c20f0a9e94e5c7d0db51ac73f4249c02d9f58c Stats: 10 lines in 1 file changed: 2 ins; 2 del; 6 mod 7904023: SHOULD NOT OCCUR: unknown cp_type Reviewed-by: shurailine ------------- PR: https://git.openjdk.org/jcov/pull/59 From jvanek at openjdk.org Mon Jun 16 16:07:16 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Mon, 16 Jun 2025 16:07:16 GMT Subject: RFR: Initialised i Message-ID: https://github.com/judovana/jcov/commit/e2c20f0a9e94e5c7d0db51ac73f4249c02d9f58c moved the usage of `i` before its declaration. As its position aftyer i+=4 was wrong, moving before ------------- Commit messages: - Initialised i Changes: https://git.openjdk.org/jcov/pull/60/files Webrev: https://webrevs.openjdk.org/?repo=jcov&pr=60&range=00 Stats: 3 lines in 1 file changed: 1 ins; 2 del; 0 mod Patch: https://git.openjdk.org/jcov/pull/60.diff Fetch: git fetch https://git.openjdk.org/jcov.git pull/60/head:pull/60 PR: https://git.openjdk.org/jcov/pull/60 From lkuskov at openjdk.org Mon Jun 16 17:51:17 2025 From: lkuskov at openjdk.org (Leonid Kuskov) Date: Mon, 16 Jun 2025 17:51:17 GMT Subject: Integrated: 7904023: SHOULD NOT OCCUR: unknown cp_type Message-ID: Fixing syntax error caused by last-minute code refactoring. ------------- Commit messages: - CODETOOLS-7904023: SHOULD NOT OCCUR: unknown cp_type Changes: https://git.openjdk.org/jcov/pull/61/files Webrev: https://webrevs.openjdk.org/?repo=jcov&pr=61&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7904023 Stats: 4 lines in 1 file changed: 2 ins; 2 del; 0 mod Patch: https://git.openjdk.org/jcov/pull/61.diff Fetch: git fetch https://git.openjdk.org/jcov.git pull/61/head:pull/61 PR: https://git.openjdk.org/jcov/pull/61 From lkuskov at openjdk.org Mon Jun 16 17:51:17 2025 From: lkuskov at openjdk.org (Leonid Kuskov) Date: Mon, 16 Jun 2025 17:51:17 GMT Subject: Integrated: 7904023: SHOULD NOT OCCUR: unknown cp_type In-Reply-To: References: Message-ID: On Mon, 16 Jun 2025 17:45:34 GMT, Leonid Kuskov wrote: > Fixing syntax error caused by last-minute code refactoring. This pull request has now been integrated. Changeset: f111dd53 Author: Leonid Kuskov URL: https://git.openjdk.org/jcov/commit/f111dd5321645ba60ca1ec0c4bed842958d4bfb2 Stats: 4 lines in 1 file changed: 2 ins; 2 del; 0 mod 7904023: SHOULD NOT OCCUR: unknown cp_type ------------- PR: https://git.openjdk.org/jcov/pull/61 From lkuskov at openjdk.org Mon Jun 16 20:37:17 2025 From: lkuskov at openjdk.org (Leonid Kuskov) Date: Mon, 16 Jun 2025 20:37:17 GMT Subject: RFR: 7904023: SHOULD NOT OCCUR: unknown cp_type Message-ID: Fixing syntax error caused by last-minute code refactoring. ------------- Commit messages: - CODETOOLS-7904023: SHOULD NOT OCCUR: unknown cp_type Changes: https://git.openjdk.org/jcov/pull/62/files Webrev: https://webrevs.openjdk.org/?repo=jcov&pr=62&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7904023 Stats: 4 lines in 1 file changed: 2 ins; 1 del; 1 mod Patch: https://git.openjdk.org/jcov/pull/62.diff Fetch: git fetch https://git.openjdk.org/jcov.git pull/62/head:pull/62 PR: https://git.openjdk.org/jcov/pull/62 From lkuskov at openjdk.org Mon Jun 16 20:41:39 2025 From: lkuskov at openjdk.org (Leonid Kuskov) Date: Mon, 16 Jun 2025 20:41:39 GMT Subject: Integrated: 7904023: SHOULD NOT OCCUR: unknown cp_type In-Reply-To: References: Message-ID: On Mon, 16 Jun 2025 20:31:49 GMT, Leonid Kuskov wrote: > Fixing syntax error caused by last-minute code refactoring. This pull request has now been integrated. Changeset: d843986b Author: Leonid Kuskov URL: https://git.openjdk.org/jcov/commit/d843986b396dd17933d5b842e64e9f336cfb5af7 Stats: 4 lines in 1 file changed: 2 ins; 1 del; 1 mod 7904023: SHOULD NOT OCCUR: unknown cp_type ------------- PR: https://git.openjdk.org/jcov/pull/62 From jvanek at openjdk.org Tue Jun 17 08:03:42 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 17 Jun 2025 08:03:42 GMT Subject: RFR: Initialised i In-Reply-To: References: Message-ID: <6IV1twKev8fgaeNH0XLzGHcQSN86_DIS-F7zahLymVo=.f399a338-e50c-41fa-a55c-97a1cc65fcf7@github.com> On Mon, 16 Jun 2025 16:02:04 GMT, Ji?? Van?k wrote: > https://github.com/judovana/jcov/commit/e2c20f0a9e94e5c7d0db51ac73f4249c02d9f58c moved the usage of `i` before its declaration. As its position aftyer i+=4 was wrong, moving before Obsoleted by https://github.com/openjdk/jcov/commit/f111dd5321645ba60ca1ec0c4bed842958d4bfb2 ------------- PR Comment: https://git.openjdk.org/jcov/pull/60#issuecomment-2979340673 From jvanek at openjdk.org Tue Jun 17 08:03:42 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 17 Jun 2025 08:03:42 GMT Subject: Withdrawn: Initialised i In-Reply-To: References: Message-ID: <6aldwO8ocpK2QnMytdJ2AKemVtcjKeYRl-UVTQPTKPU=.4ef02616-38b1-43cb-9ced-a43fdcf13ef9@github.com> On Mon, 16 Jun 2025 16:02:04 GMT, Ji?? Van?k wrote: > https://github.com/judovana/jcov/commit/e2c20f0a9e94e5c7d0db51ac73f4249c02d9f58c moved the usage of `i` before its declaration. As its position aftyer i+=4 was wrong, moving before This pull request has been closed without being integrated. ------------- PR: https://git.openjdk.org/jcov/pull/60 From shurailine at openjdk.org Mon Jun 30 22:52:25 2025 From: shurailine at openjdk.org (Alexandre Iline) Date: Mon, 30 Jun 2025 22:52:25 GMT Subject: RFR: 7904052: Allow JCov tests to be run by JTReg Message-ID: JCov provides better isolation and better control over the execution environment. For now I kept the testng test subject, up for discussion. ------------- Commit messages: - And the target to run tests - CODETOOLS-7904052: Allow JCov tests to be run by JTReg Changes: https://git.openjdk.org/jcov/pull/63/files Webrev: https://webrevs.openjdk.org/?repo=jcov&pr=63&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7904052 Stats: 13 lines in 2 files changed: 13 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jcov/pull/63.diff Fetch: git fetch https://git.openjdk.org/jcov.git pull/63/head:pull/63 PR: https://git.openjdk.org/jcov/pull/63