[jdk21u-dev] RFR: 8324678: Replace NULL with nullptr in HotSpot gtests [v2]

Goetz Lindenmaier goetz at openjdk.org
Wed Mar 12 21:20:33 UTC 2025


> test/hotspot/gtest/compiler/test_directivesParser.cpp
> 8327986: ASAN reports use-after-free in DirectivesParserTest.empty_object_vm
> was already backported.
> Failed chunk not applicable, skipped.
> 
> test/hotspot/gtest/gc/shared/test_bufferNodeAllocator.cpp
> History of this file differs in a row of changes.
> But here only Copyright needs to be resolved.
> 
> I backport this for parity with 21.0.8-oracle.
> 
> I don't really see why this was backported. But let's go 
> along anyways, this is test-only coding and just a bit
> tedious to prepare. But it might make later backports
> much easier.
> 
> Resolved files in detail:
> 
> test/hotspot/gtest/jfr/test_adaptiveSampler.cpp
> 8313552: Fix -Wconversion warnings in JFR code
> missing in 21.  This change sounds like a nice-to-have in 21,
> but it is large.
> 
> test/hotspot/gtest/logging/test_logConfiguration.cpp
> test/hotspot/gtest/logging/test_logTagSet.cpp
> 8299825: Move StdoutLog and StderrLog to LogConfiguration
> missing in 21. Adapted all remaining NULLs.
> 
> 
> test/hotspot/gtest/metaspace/test_binlist.cpp
> 8307356: Metaspace: simplify BinList handling
> missing in 21. Not a candidate for backport.
> One trivial resolve.
> 
> 
> test/hotspot/gtest/metaspace/test_metaspacearena_stress.cpp
> 8307356: Metaspace: simplify BinList handling
> 8314654: Metaspace: move locking out of MetaspaceArena
> missing in 21. Not candidates for backport.
> Resolved and flipped one additional NULL.
> 
> test/hotspot/gtest/runtime/test_os_linux.cpp
> 8261894: Remove support for UseSHM
> not in 21 and not a candidate, and later change
> 8341722: Fix some warnings as errors when building on Linux with toolchain clang
> was already backported.
> Resolved all remaining NULLs.

Goetz Lindenmaier has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:

  Backport c1281e6b45ed167df69d29a6039d81854c145ae6

-------------

Changes:
  - all: https://git.openjdk.org/jdk21u-dev/pull/1476/files
  - new: https://git.openjdk.org/jdk21u-dev/pull/1476/files/aa4ca4cc..b59d98ab

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=1476&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=1476&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk21u-dev/pull/1476.diff
  Fetch: git fetch https://git.openjdk.org/jdk21u-dev.git pull/1476/head:pull/1476

PR: https://git.openjdk.org/jdk21u-dev/pull/1476


More information about the jdk-updates-dev mailing list