March 24, 2026
7:26 p.m.
On Mon Mar 23 23:45:18 2026 +0000, Jacek Caban wrote:
While it seems that we should indeed do that in this case, there are more cases with the same problem. Could we do that in `variant_call`, or even in `disp_call`, instead? Added some more test cases.
Moving the fix to variant_call doesn't work because variant_call's VT_BYREF|VT_VARIANT return is correct and needed for the do_icall path — tests like getVT(arr(1)) expect "VT_EMPTY*" (a byref), which matches Windows behavior. Changing variant_call to always copy breaks these tests. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10362#note_133563