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; ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3391#note_40343