24 Apr
2025
24 Apr
'25
8:32 p.m.
On Thu Apr 24 19:15:13 2025 +0000, Jacek Caban wrote:
It looks like `native_function_string` doesn't really expect a NULL name, it ends up adding an extra space in the prefix. Seems like we're missing a test for this case. Other than that, looks good to me. To be fair it looks like it's actually wrong anyway, it should use the constructor's name. I made it work on mshtml side now, let me know if I should move it to jscript, the only issue I see is that the dispex name is in ASCII (due to gecko), so we'd need to convert it to WCHAR somehow without allocating, probably in a buffer? I think it's better this way though.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/7779#note_101822