Re: [PATCH v17 0/2] MR3391: mshtml: implement window.MutationObserver with MutationObserver stub

[Gabriel Ivăncescu (＠insn)]

On Wed Jul 26 19:04:00 2023 +0000, Yuxuan Shui wrote:
> We have named arg `this`.

I see, you're right, but I don't think that code is handling it correctly, it implicitly assumes it is DISPID_THIS. I think this should be checked, then, with a FIXME if it's not. So if cNamedArgs is > 1 or if it's 1 but not DISPID_THIS, print FIXME and return E_NOTIMPL (we have many cases of this in dispex.c). And then I guess you can use `argc` again. Sorry for the confusion earlier.