RFR: 8286117: Remove unnecessary indirection and unused code in UL
Johan Sjölén
duke at openjdk.java.net
Wed May 4 18:35:52 UTC 2022
This PR cleans up some minor annoyances in the UL code.
Note that the change from `LogImpl` to `LogTagSet` is equivalent, we're essentially inlining the calls that `LogImpl` would have made. This simplifies understanding ("why do we need `LogImpl` here?" was my question, answer: We don't!)'
-------------
Commit messages:
- Use uniform initialization syntax in top level
- Use LogTagSet directly in LogMessage
- Use LogTagSetMapping in LogTargetImpl instead of LogImpl
- Declare pointers to LogTagSet const
- Fix comments in LogStream
Changes: https://git.openjdk.java.net/jdk/pull/8541/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=8541&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8286117
Stats: 18 lines in 5 files changed: 0 ins; 3 del; 15 mod
Patch: https://git.openjdk.java.net/jdk/pull/8541.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/8541/head:pull/8541
PR: https://git.openjdk.java.net/jdk/pull/8541
More information about the hotspot-runtime-dev
mailing list