Gabriel Ivăncescu (@insn) commented about dlls/jscript/array.c:
> }
>
> add_args = argc-2;
> + } else if (argc && ctx->html_mode) {
> + delete_cnt = length-start;
Are you sure this is html mode specific? To confirm, you should move the test to `documentmode.js` and see what happens in every mode, `es5.js` is for ES5+ (IE9+, but we test it in IE11 mode only).
If it turns out to be ES5 for example, you can use ctx->version >= SCRIPTLANGUAGEVERSION_ES5 instead. Otherwise you can keep html mode of course, if it works in all modes the same.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3567#note_42308
> Actually there is some logic, and NULL handle has a special meaning in NtTerminateProcess.
Thanks. I forgot to articulate it even as I noticed that peculiarity. That's on me.
> This would need tests to determine if the special meaning also applies to NtTerminateThread.
The most obvious test that I can think off the top of my head is whether `NtTerminateThread(NULL, _)` returns or not. Also perhaps we can copy some applicable tests from `NtTerminateProcess(NULL, _)` as well. What other tests should be done, though?
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3447#note_42305