RFR: 8299825: Move StdoutLog and StderrLog to LogConfiguration

Johan Sjölen jsjolen at openjdk.org
Thu Jun 8 11:59:54 UTC 2023


Hi,

Please consider this small cleanup. `LogConfiguration` initializes and handles the lifetime of `StdoutLog` and `StderrLog`, therefore they should logically belong to that class. I leave the visibility as public for the time being, to change it to private would require the gtests to all be part of a fixture.

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

Commit messages:
 - Restore syntactic alignment
 - Move StdoutLog and StderrLog into LogConfiguration

Changes: https://git.openjdk.org/jdk/pull/14376/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=14376&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8299825
  Stats: 54 lines in 6 files changed: 8 ins; 2 del; 44 mod
  Patch: https://git.openjdk.org/jdk/pull/14376.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14376/head:pull/14376

PR: https://git.openjdk.org/jdk/pull/14376


More information about the hotspot-runtime-dev mailing list