On 26/04/2024 09:39, Torbjorn SVENSSON wrote: > 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
Feel free to post some additional tests, Torbjorn. R.