> 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:
Apply suggestions from code review
Co-authored-by: Alexey Ivanov <alexey.iva...@oracle.com>
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/24949/files
- new: https://git.openjdk.org/jdk/pull/24949/files/867aee60..ad71f014
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=24949&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=24949&range=01-02
Stats: 19 lines in 9 files changed: 2 ins; 0 del; 17 mod
Patch: https://git.openjdk.org/jdk/pull/24949.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/24949/head:pull/24949
PR: https://git.openjdk.org/jdk/pull/24949
