RFR: 8310405: Linker.Option.firstVariadicArg should specify which index values are valid

Per Minborg pminborg at openjdk.org
Wed Jun 21 07:37:04 UTC 2023


On Wed, 21 Jun 2023 00:07:54 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

> Possible suggestion/thing to try: use a bullet list to spell out all cases for `index`. E.g. we know there's index == 0 (all variadic). Then we know there's index = N (no variadic). Then there's index == m, 0 < m < N - which means layouts 0..m are non-variadic and m..N are variadic (where n..m denotes an interval with n included and m excluded).

I think this is a good suggestion. It makes it much easier to understand.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14565#discussion_r1236509508


More information about the core-libs-dev mailing list