[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-serviceability-dev
Subject: Integrated: 8298241: Replace C-style casts with JavaThread::cast
From: David Holmes <dholmes () openjdk ! org>
Date: 2022-12-15 21:20:32
Message-ID: lPTq5YGLAbONseoFbglraWLSYLRoM-sVS3NsbHm6PTI=.1706b724-bf2e-4a36-b311-18cc89ebc0b0 () github ! com
[Download RAW message or body]
On Wed, 14 Dec 2022 22:22:48 GMT, David Holmes <dholmes@openjdk.org> wrote:
> This is a simple cleanup RFE to get rid of old-style C casts in relation to \
> JavaThread.
> In many cases involving NULL/nullptr the cast could just be dropped. Sometimes a \
> static cast is needed to disambiguate overloads.
> A couple of reinterpret_cast are needed when doing int/ptr conversion.
>
> static_cast is used for void* conversion.
>
> The other changes should be self explanatory.
>
> The changes in
>
> src/hotspot/os_cpu/bsd_aarch64/javaThread_bsd_aarch64.cpp
> src/hotspot/os_cpu/windows_aarch64/javaThread_windows_aarch64.cpp
>
> are a bit more extensive. That code was using an alias for `this` which is \
> completely unnecessary (and the alias creation was using the cast). This could be \
> factored out if thought necessary.
> I grep'd as best I could for the old C-style casts but can't be certain I got them \
> all.
> Testing:
> - all builds in our tiers1-5
> - local linux x64 product, slowdebug and fastdebug
> - GHA
> - Sanity testing tiers 1-3
> Thanks.
This pull request has now been integrated.
Changeset: 3cdbd878
Author: David Holmes <dholmes@openjdk.org>
URL: https://git.openjdk.org/jdk/commit/3cdbd878e68dc1131093137a7357710ad303ae8c
Stats: 44 lines in 19 files changed: 0 ins; 11 del; 33 mod
8298241: Replace C-style casts with JavaThread::cast
Reviewed-by: coleenp, stefank, sspitsyn
-------------
PR: https://git.openjdk.org/jdk/pull/11682
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic