Integrated: 8299825: Move StdoutLog and StderrLog to LogConfiguration

Johan Sjölen jsjolen at openjdk.org
Wed Jun 28 13:34:17 UTC 2023


On Thu, 8 Jun 2023 11:52:11 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: 2ccdd292
Author:    Johan Sjölen <jsjolen at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/2ccdd29206d5041b067fac4e3c3b248e8d0aa3df
Stats:     49 lines in 7 files changed: 8 ins; 5 del; 36 mod

8299825: Move StdoutLog and StderrLog to LogConfiguration

Reviewed-by: dholmes, azafari

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

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


More information about the hotspot-runtime-dev mailing list