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

[Gabriel Ivăncescu (＠insn)]

Gabriel Ivăncescu (@insn) commented about dlls/mshtml/mutation.c:
> -
> - TRACE("(%p)->(%lx %x %p %p %p %p)\n", This, lcid, flags, params, res, ei, caller);
> -
> - if (argc < 1)
> -
>    return E_UNEXPECTED;
> -
> - callback = params->rgvarg + (params->cArgs - 1);
> - if (V_VT(callback) != VT_DISPATCH) {
> -
>    FIXME("Should return TypeMismatchError\n");
> -
>    return E_FAIL;
> - }
> -
> - if (!res) {
> -
>    /* This function has no side effects if the return value is dropped. */
> -
>    return S_OK;
> - }

This comment doesn't serve much purpose; this is just an optimization (technically, one could return the object, and then it gets immediately discarded, which has the same effect). So:

```suggestion:-3+0
    if (!res)
        return S_OK;
```