RFR: 8311823: Remove JFR: Uninitialized EventEmitter::_thread_id field
[JDK-8311823](https://bugs.openjdk.org/browse/JDK-8311823) Remove Uninitialized EventEmitter::_thread_id field from src/hotspot/share/jfr/leakprofiler/checkpoint/eventEmitter.hpp GHA tests pass Additional testing: make test TEST=jdk/jfr ------------- Commit messages: - 8311823: Remove JFR: Uninitialized EventEmitter::_ Changes: https://git.openjdk.org/jdk/pull/15893/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=15893&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8311823 Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod Patch: https://git.openjdk.org/jdk/pull/15893.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/15893/head:pull/15893 PR: https://git.openjdk.org/jdk/pull/15893
On Fri, 22 Sep 2023 20:53:27 GMT, Elif Aslan <duke@openjdk.org> wrote:
[JDK-8311823](https://bugs.openjdk.org/browse/JDK-8311823)
Remove Uninitialized EventEmitter::_thread_id field from src/hotspot/share/jfr/leakprofiler/checkpoint/eventEmitter.hpp
GHA tests pass
Additional testing: make test TEST=jdk/jfr
Looks fine. This is a first contribution, so I would like to have another reviewer as well. ------------- Marked as reviewed by shade (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/15893#pullrequestreview-1642314052
On Fri, 22 Sep 2023 20:53:27 GMT, Elif Aslan <duke@openjdk.org> wrote:
[JDK-8311823](https://bugs.openjdk.org/browse/JDK-8311823)
Remove Uninitialized EventEmitter::_thread_id field from src/hotspot/share/jfr/leakprofiler/checkpoint/eventEmitter.hpp
GHA tests pass
Additional testing: make test TEST=jdk/jfr
@egahlin would like to quickly approve this, maybe? ------------- PR Comment: https://git.openjdk.org/jdk/pull/15893#issuecomment-1733863867
On Fri, 22 Sep 2023 20:53:27 GMT, Elif Aslan <duke@openjdk.org> wrote:
[JDK-8311823](https://bugs.openjdk.org/browse/JDK-8311823)
Remove Uninitialized EventEmitter::_thread_id field from src/hotspot/share/jfr/leakprofiler/checkpoint/eventEmitter.hpp
GHA tests pass
Additional testing: make test TEST=jdk/jfr
Marked as reviewed by egahlin (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/15893#pullrequestreview-1642352088
On Fri, 22 Sep 2023 20:53:27 GMT, Elif Aslan <duke@openjdk.org> wrote:
[JDK-8311823](https://bugs.openjdk.org/browse/JDK-8311823)
Remove Uninitialized EventEmitter::_thread_id field from src/hotspot/share/jfr/leakprofiler/checkpoint/eventEmitter.hpp
GHA tests pass
Additional testing: make test TEST=jdk/jfr
This pull request has now been integrated. Changeset: 9688ec2d Author: Elif Aslan <elifasln@amazon.com> Committer: Aleksey Shipilev <shade@openjdk.org> URL: https://git.openjdk.org/jdk/commit/9688ec2d0d9fd830d993454ea093b68b0f85ebe7 Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod 8311823: JFR: Uninitialized EventEmitter::_thread_id field Reviewed-by: shade, egahlin ------------- PR: https://git.openjdk.org/jdk/pull/15893
participants (3)
-
Aleksey Shipilev
-
Elif Aslan
-
Erik Gahlin