> 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 two additional
commits since the last revision:
- fixup! 8355904: Use variadic macros for J2dTrace
- fixup! 8355904: Use variadic macros for J2dTrace
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/24949/files
- new: https://git.openjdk.org/jdk/pull/24949/files/ad71f014..df5781f7
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=24949&range=03
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=24949&range=02-03
Stats: 670 lines in 64 files changed: 63 ins; 4 del; 603 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
