RFR: 8286117: Remove unnecessary indirection and unused code in UL [v2]

David Holmes dholmes at openjdk.java.net
Mon May 9 12:54:53 UTC 2022


On Mon, 9 May 2022 11:34:43 GMT, Johan Sjölén <duke at openjdk.java.net> wrote:

>> So really `_log` is just a simple alias that makes the other code easier to read IMO. So can't we keep `_log` and change the type?
>
> How about `LogTagSet& _tagset`? There are more examples of `_tagset` members in UL, so it's consistent.

That seems a bit odd to me. Why can't we use the actual thing we will log to? The description of LogMessage is that it is a multi-part log entry that is accumulated until we flush it by writing to the actual log device.

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

PR: https://git.openjdk.java.net/jdk/pull/8541


More information about the hotspot-runtime-dev mailing list