On Tue, 18 Nov 2025 16:41:40 GMT, Sean Coffey <[email protected]> wrote:

>> Since you have now imported `@ForceInline` maybe you could also add it to 
>> `doLog` - the methods that call it are all one-liners.
>
> log0 has quite a bit of logic in it. Not sure if it warrants inlining, The 
> isOn() check will short circuit nearly all log calls in the default JDK 
> configuration in any case.

> `log0` might be a better name than `doLog`

thanks - made that change

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28376#discussion_r2538924890

Reply via email to