[jdk17u-dev] Integrated: 8298730: Refactor subsystem_file_line_contents and add docs and tests
Goetz Lindenmaier
goetz at openjdk.org
Thu Apr 3 11:44:01 UTC 2025
On Mon, 31 Mar 2025 13:05:13 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:
> I backport this for parity with 17.0.16-oracle.
>
> I had to resolve one larger chunk in cgroupSubsystem_linux.hpp. Changes [8301479: Replace NULL with nullptr in os/linux](https://github.com/openjdk/jdk21u-dev/commit/ac9e046748a9bb6ee065dc473d82135ce36043b7) and [8238161: use os::fopen in HS code where possible](https://github.com/openjdk/jdk21u-dev/commit/f54ce84474c2ced340c92564814fa5c221415944) are not in 17.
>
> I did add the new code as-is wrt. nullptr. but kept the call to fopen as it was in 17 before this change.
>
> To make it compile I add changes to ostream.hpp from https://bugs.openjdk.org/browse/JDK-8293251: Use stringStream::base() instead of as_string() when applicable
> See extra commit.
This pull request has now been integrated.
Changeset: bb103d7d
Author: Goetz Lindenmaier <goetz at openjdk.org>
URL: https://git.openjdk.org/jdk17u-dev/commit/bb103d7d0914f90e85eb5f87831ffef1de536072
Stats: 293 lines in 7 files changed: 232 ins; 15 del; 46 mod
8298730: Refactor subsystem_file_line_contents and add docs and tests
Reviewed-by: phh
Backport-of: 500c3c17379fe0a62d42ba31bdcdb584b1823f60
-------------
PR: https://git.openjdk.org/jdk17u-dev/pull/3411
More information about the jdk-updates-dev
mailing list