Hi,
On 2024-04-25 16:25, Richard Ball wrote:
Hi Torbjorn,
Thanks very much for the comments.
I think given that the code that handles this, is within a
FOREACH_FUNCTION_ARGS loop.
It seems a fairly safe assumption that if the code works for one that it
will work for all.
To go back and add extra tests to me seems a little overkill.
For verifying that the implementation does the right thing now, no, but
for verifying against future regressions, then yes.
So, from a regression point of view, I think it makes sense to have the
check that more than the first argument is managed properly.
Kind regards,
Torbjörn