Integrated: 8358588: ThreadSnapshot.ThreadLock should be static nested class
Aleksey Shipilev
shade at openjdk.org
Thu Jun 5 09:04:55 UTC 2025
On Wed, 4 Jun 2025 12:03:15 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> SonarCloud points out that ThreadLock class introduced by [JDK-8357650](https://bugs.openjdk.org/browse/JDK-8357650) can be turned into static nested class. I don't think this shows any real bug yet, as unreferenced enclosing class reference gets nowhere. But it would be nice to be extra crisp here.
>
> Additional testing:
> - [x] Linux x86_64 server fastdebug, `serviceability/`
> - [x] Linux x86_64 server fastdebug, `jdk_management`
This pull request has now been integrated.
Changeset: dc949003
Author: Aleksey Shipilev <shade at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/dc949003ded278805d10c7b630e82348a7d998fe
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
8358588: ThreadSnapshot.ThreadLock should be static nested class
Reviewed-by: alanb, sspitsyn, amenkov
-------------
PR: https://git.openjdk.org/jdk/pull/25638
More information about the serviceability-dev
mailing list