On Tue Aug 15 20:38:27 2023 +0000, Gabriel Ivăncescu wrote:
Ok new version should just do more prep work and also convert nodes at the end to use the dispex CC instead. No other objects are done yet in this MR. Note that I went with renamed versions of original helpers as I think it's cleaner, since eventually we'll get rid of them (when we have CC for every object), so this provides some progress indication rather than renaming them at the end.
FWIW, I'm not sure if `dispex_query_interface` interface is a final thing, but that's not something we need to worry now. Note that once your conversion is done, we could just forward `AddRef` and `Release` to IDispatchEx like we do for all other IDispatch function. If we did the same to `QueryInterface`, similar to how `HTMLDOMNode` does it with its vtbl, we could even get rid of outer pointer from DispatchEx. (outer was needed for HTMLDocumentObj in the past, but it's no longer the case).
Anyway, those are just some thoughts for the future, I approved the MR.