RFR: 8343513: Forward declare Thread in mutexLocker.hpp [v2]
Johan Sjölen
jsjolen at openjdk.org
Mon Nov 4 08:32:02 UTC 2024
> Hi,
>
> Let's explicitly forward declare `Thread` in `mutexLocker.hpp` instead of including `thread.hpp`. Previously we got the forward declaration from `memory/allocation.hpp`, but let's be explicit about forward declarations being used when we can.
>
> Thanks
Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision:
Also requires NMT to actually include thread
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/21868/files
- new: https://git.openjdk.org/jdk/pull/21868/files/d019b556..b45dc667
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=21868&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=21868&range=00-01
Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/21868.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/21868/head:pull/21868
PR: https://git.openjdk.org/jdk/pull/21868
More information about the hotspot-runtime-dev
mailing list