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

Reply via email to