On Tue, 29 Apr 2025 14:04:07 GMT, Nikita Gubarkov <ngubar...@openjdk.org> wrote:
>> J2dTrace macros have multiple overloads specifying number of arguments,
>> making it less convent to change number of arguments. There were cases when
>> existing macros were not enough and people had to add new variants with even
>> more arguments. We could simply use variadic macros instead.
>>
>> Also, currently those macros expand to a { code block }, which doesn't
>> require a semicolon at the end, so it can sometimes be missed, leading to an
>> inconsistent code style. We could expand it directly to the function,
>> forcing user to insert a semicolon after that, in a function-like style.
>
> Nikita Gubarkov has updated the pull request incrementally with one
> additional commit since the last revision:
>
> fixup! 8355904: Use variadic macros for J2dTrace
LGTM
-------------
Marked as reviewed by avu (Committer).
PR Review: https://git.openjdk.org/jdk/pull/24949#pullrequestreview-2832198050
