August 2023 - Wine-commits - list.winehq.org

Jeff Smith : msvcp120/tests: Use compare_uint() in compare_float() instead of abs().
by Alexandre Julliard 08 Aug '23

08 Aug '23

Module: wine Branch: master Commit: ed29473e60ad4e6601b12bbeb37ad56156fa9e42 URL: https://gitlab.winehq.org/wine/wine/-/commit/ed29473e60ad4e6601b12bbeb37ad5… Author: Jeff Smith <whydoubt(a)gmail.com> Date: Wed Aug 2 22:19:19 2023 -0500 msvcp120/tests: Use compare_uint() in compare_float() instead of abs(). The result of abs(INT_MIN) is INT_MIN, which breaks the ulps comparison. --- dlls/msvcp120/tests/msvcp120.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/dlls/msvcp120/tests/msvcp120.c b/dlls/msvcp120/tests/msvcp120.c index 44c0a3f0fe0..060dda5b45f 100644 --- a/dlls/msvcp120/tests/msvcp120.c +++ b/dlls/msvcp120/tests/msvcp120.c @@ -187,6 +187,13 @@ enum file_type { type_unknown }; +static BOOL compare_uint(unsigned int x, unsigned int y, unsigned int max_diff) +{ + unsigned int diff = x > y ? x - y : y - x; + + return diff <= max_diff; +} + static BOOL compare_float(float f, float g, unsigned int ulps) { int x = *(int *)&f; @@ -197,10 +204,7 @@ static BOOL compare_float(float f, float g, unsigned int ulps) if (y < 0) y = INT_MIN - y; - if (abs(x - y) > ulps) - return FALSE; - - return TRUE; + return compare_uint(x, y, ulps); } static char* (__cdecl *p_setlocale)(int, const char*);

1 0

Jinoh Kang : ntdll: Avoid relying on linux/ipx.h to define SOL_IPX.
by Alexandre Julliard 08 Aug '23

08 Aug '23

Module: wine Branch: master Commit: 14d4a896bd60d722aa6fed9bbedc39cc36036bef URL: https://gitlab.winehq.org/wine/wine/-/commit/14d4a896bd60d722aa6fed9bbedc39… Author: Jinoh Kang <jinoh.kang.kr(a)gmail.com> Date: Thu Jul 27 22:39:50 2023 +0900 ntdll: Avoid relying on linux/ipx.h to define SOL_IPX. musl libc doesn't supply any definitions for IPX, such as the SOL_IPX macro. However, it still provides linux/ipx.h from Linux uAPI header files if it exists. Linux kernel wouldn't drop linux/ipx.h from uAPI headers until 5.15, although IPX support has already been marked obsolete since 2018. Fix this by not defining HAS_IPX if linux/ipx.h has been included but nothing defines the SOL_IPX macro. Status of IPX support from other libcs are noted below: - bionic: netipx/ipx.h does not exist. linux/ipx.h may or may not exist. Note that sys/socket.h defines SOL_IPX even if linux/ipx.h is missing. - glibc: netipx/ipx.h exists. In this case, Wine assumes IPX support even if the operating system does not support it in runtime. - BSD variants: netipx/ipx.h may or may not exist. linux/ipx.h does not exist. Some BSDs supply SO_DEFAULT_HEADERS instead of SOL_IPX. Fixes: 954bf9e0505a7368c34047b311d613cff04393a1 --- dlls/ntdll/unix/socket.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/dlls/ntdll/unix/socket.c b/dlls/ntdll/unix/socket.c index 6969f411955..e09bf34b07d 100644 --- a/dlls/ntdll/unix/socket.c +++ b/dlls/ntdll/unix/socket.c @@ -54,7 +54,9 @@ # include <linux/types.h> # endif # include <linux/ipx.h> -# define HAS_IPX +# ifdef SOL_IPX +# define HAS_IPX +# endif #endif #ifdef HAVE_LINUX_IRDA_H

1 0

Jinoh Kang : ws2_32: Avoid relying on linux/ipx.h to define SOL_IPX.
by Alexandre Julliard 08 Aug '23

08 Aug '23

Module: wine Branch: master Commit: 0deab78dc5ea69245c73d08ac7da52c9cef8313d URL: https://gitlab.winehq.org/wine/wine/-/commit/0deab78dc5ea69245c73d08ac7da52… Author: Jinoh Kang <jinoh.kang.kr(a)gmail.com> Date: Thu Jul 27 22:39:50 2023 +0900 ws2_32: Avoid relying on linux/ipx.h to define SOL_IPX. musl libc doesn't supply any definitions for IPX, such as the SOL_IPX macro. However, it still provides linux/ipx.h from Linux uAPI header files if it exists. Linux kernel wouldn't drop linux/ipx.h from uAPI headers until 5.15, although IPX support has already been marked obsolete since 2018. Fix this by not defining HAS_IPX if linux/ipx.h has been included but nothing defines the SOL_IPX macro. Status of IPX support from other libcs are noted below: - bionic: netipx/ipx.h does not exist. linux/ipx.h may or may not exist. Note that sys/socket.h defines SOL_IPX even if linux/ipx.h is missing. - glibc: netipx/ipx.h exists. In this case, Wine assumes IPX support even if the operating system does not support it in runtime. - BSD variants: netipx/ipx.h may or may not exist. linux/ipx.h does not exist. Some BSDs supply SO_DEFAULT_HEADERS instead of SOL_IPX. Fixes: da4e359a700209c840490483f5337a14aa1d448f --- dlls/ws2_32/unixlib.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/dlls/ws2_32/unixlib.c b/dlls/ws2_32/unixlib.c index 5cb64aad258..db36f18c7bd 100644 --- a/dlls/ws2_32/unixlib.c +++ b/dlls/ws2_32/unixlib.c @@ -75,7 +75,9 @@ # include <linux/types.h> # endif # include <linux/ipx.h> -# define HAS_IPX +# ifdef SOL_IPX +# define HAS_IPX +# endif #endif #ifdef HAVE_LINUX_IRDA_H

1 0

Jinoh Kang : server: Avoid relying on linux/ipx.h to define SOL_IPX.
by Alexandre Julliard 08 Aug '23

08 Aug '23

Module: wine Branch: master Commit: 915c3910143a83913db8d9bed8e8c97edc863beb URL: https://gitlab.winehq.org/wine/wine/-/commit/915c3910143a83913db8d9bed8e8c9… Author: Jinoh Kang <jinoh.kang.kr(a)gmail.com> Date: Thu Jul 27 22:39:50 2023 +0900 server: Avoid relying on linux/ipx.h to define SOL_IPX. musl libc doesn't supply any definitions for IPX, such as the SOL_IPX macro. However, it still provides linux/ipx.h from Linux uAPI header files if it exists. Linux kernel wouldn't drop linux/ipx.h from uAPI headers until 5.15, although IPX support has already been marked obsolete since 2018. Fix this by not defining HAS_IPX if linux/ipx.h has been included but nothing defines the SOL_IPX macro. Status of IPX support from other libcs are noted below: - bionic: netipx/ipx.h does not exist. linux/ipx.h may or may not exist. Note that sys/socket.h defines SOL_IPX even if linux/ipx.h is missing. - glibc: netipx/ipx.h exists. In this case, Wine assumes IPX support even if the operating system does not support it in runtime. - BSD variants: netipx/ipx.h may or may not exist. linux/ipx.h does not exist. Some BSDs supply SO_DEFAULT_HEADERS instead of SOL_IPX. Fixes: 41cc117b3f37ab4b9b4ac8a815cd2a496d38fb4b --- server/sock.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/server/sock.c b/server/sock.c index 550fe61e477..ce95b7f2998 100644 --- a/server/sock.c +++ b/server/sock.c @@ -71,7 +71,9 @@ # include <linux/types.h> # endif # include <linux/ipx.h> -# define HAS_IPX +# ifdef SOL_IPX +# define HAS_IPX +# endif #endif #ifdef HAVE_LINUX_IRDA_H

1 0

Gabriel Ivăncescu : mshtml: Use unlink and destructor in the vtbl for Console.
by Alexandre Julliard 08 Aug '23

08 Aug '23

Module: wine Branch: master Commit: a916bc768d8bb281e2ad67e58ba9839b36e850b4 URL: https://gitlab.winehq.org/wine/wine/-/commit/a916bc768d8bb281e2ad67e58ba983… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Fri Aug 4 17:30:14 2023 +0300 mshtml: Use unlink and destructor in the vtbl for Console. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- dlls/mshtml/omnavigator.c | 21 +++++++++++++++++---- 1 file changed, 17 insertions(+), 4 deletions(-) diff --git a/dlls/mshtml/omnavigator.c b/dlls/mshtml/omnavigator.c index f6cf0755a74..71beb3cc41c 100644 --- a/dlls/mshtml/omnavigator.c +++ b/dlls/mshtml/omnavigator.c @@ -2609,10 +2609,8 @@ static ULONG WINAPI console_Release(IWineMSHTMLConsole *iface) TRACE("(%p) ref=%ld\n", console, ref); - if(!ref) { + if(!ref) release_dispex(&console->dispex); - free(console); - } return ref; } @@ -2789,13 +2787,28 @@ static const IWineMSHTMLConsoleVtbl WineMSHTMLConsoleVtbl = { console_warn, }; +static inline struct console *console_from_DispatchEx(DispatchEx *iface) +{ + return CONTAINING_RECORD(iface, struct console, dispex); +} + +static void console_destructor(DispatchEx *dispex) +{ + struct console *console = console_from_DispatchEx(dispex); + free(console); +} + +static const dispex_static_data_vtbl_t console_dispex_vtbl = { + console_destructor, +}; + static const tid_t console_iface_tids[] = { IWineMSHTMLConsole_tid, 0 }; static dispex_static_data_t console_dispex = { L"Console", - NULL, + &console_dispex_vtbl, IWineMSHTMLConsole_tid, console_iface_tids };

1 0

Gabriel Ivăncescu : mshtml: Use unlink and destructor in the vtbl for MediaQueryList.
by Alexandre Julliard 08 Aug '23

08 Aug '23

Module: wine Branch: master Commit: 7bf4cd17ea8fb8c3487f31c093f00c78f619b554 URL: https://gitlab.winehq.org/wine/wine/-/commit/7bf4cd17ea8fb8c3487f31c093f00c… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Fri Aug 4 17:30:14 2023 +0300 mshtml: Use unlink and destructor in the vtbl for MediaQueryList. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- dlls/mshtml/omnavigator.c | 45 +++++++++++++++++++++++++++++++++------------ 1 file changed, 33 insertions(+), 12 deletions(-) diff --git a/dlls/mshtml/omnavigator.c b/dlls/mshtml/omnavigator.c index 71beb3cc41c..2e098bf2d09 100644 --- a/dlls/mshtml/omnavigator.c +++ b/dlls/mshtml/omnavigator.c @@ -2890,21 +2890,11 @@ static ULONG WINAPI media_query_list_Release(IWineMSHTMLMediaQueryList *iface) { struct media_query_list *media_query_list = impl_from_IWineMSHTMLMediaQueryList(iface); LONG ref = InterlockedDecrement(&media_query_list->ref); - struct media_query_list_listener *listener, *listener2; TRACE("(%p) ref=%ld\n", media_query_list, ref); - if(!ref) { - media_query_list->callback->media_query_list = NULL; - LIST_FOR_EACH_ENTRY_SAFE(listener, listener2, &media_query_list->listeners, struct media_query_list_listener, entry) { - IDispatch_Release(listener->function); - free(listener); - } - nsIDOMMediaQueryListListener_Release(&media_query_list->callback->nsIDOMMediaQueryListListener_iface); - nsIDOMMediaQueryList_Release(media_query_list->nsquerylist); + if(!ref) release_dispex(&media_query_list->dispex); - free(media_query_list); - } return ref; } @@ -3125,13 +3115,44 @@ static const nsIDOMMediaQueryListListenerVtbl media_query_list_callback_vtbl = { media_query_list_callback_HandleChange }; +static inline struct media_query_list *media_query_list_from_DispatchEx(DispatchEx *iface) +{ + return CONTAINING_RECORD(iface, struct media_query_list, dispex); +} + +static void media_query_list_unlink(DispatchEx *dispex) +{ + struct media_query_list *media_query_list = media_query_list_from_DispatchEx(dispex); + + media_query_list->callback->media_query_list = NULL; + while(!list_empty(&media_query_list->listeners)) { + struct media_query_list_listener *listener = LIST_ENTRY(list_head(&media_query_list->listeners), struct media_query_list_listener, entry); + list_remove(&listener->entry); + IDispatch_Release(listener->function); + free(listener); + } + unlink_ref(&media_query_list->nsquerylist); +} + +static void media_query_list_destructor(DispatchEx *dispex) +{ + struct media_query_list *media_query_list = media_query_list_from_DispatchEx(dispex); + nsIDOMMediaQueryListListener_Release(&media_query_list->callback->nsIDOMMediaQueryListListener_iface); + free(media_query_list); +} + +static const dispex_static_data_vtbl_t media_query_list_dispex_vtbl = { + media_query_list_destructor, + media_query_list_unlink +}; + static const tid_t media_query_list_iface_tids[] = { IWineMSHTMLMediaQueryList_tid, 0 }; static dispex_static_data_t media_query_list_dispex = { L"MediaQueryList", - NULL, + &media_query_list_dispex_vtbl, IWineMSHTMLMediaQueryList_tid, media_query_list_iface_tids };

1 0

Gabriel Ivăncescu : mshtml: Use unlink and destructor in the vtbl for MutationObserver.
by Alexandre Julliard 08 Aug '23

08 Aug '23

Module: wine Branch: master Commit: ce02f4d98a93d2c94cf9ebefed9cb053aa231e44 URL: https://gitlab.winehq.org/wine/wine/-/commit/ce02f4d98a93d2c94cf9ebefed9cb0… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Fri Aug 4 17:30:14 2023 +0300 mshtml: Use unlink and destructor in the vtbl for MutationObserver. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- dlls/mshtml/mutation.c | 29 ++++++++++++++++++++++++----- 1 file changed, 24 insertions(+), 5 deletions(-) diff --git a/dlls/mshtml/mutation.c b/dlls/mshtml/mutation.c index 3f544e0a90f..13d0b757158 100644 --- a/dlls/mshtml/mutation.c +++ b/dlls/mshtml/mutation.c @@ -1127,11 +1127,8 @@ static ULONG WINAPI MutationObserver_Release(IWineMSHTMLMutationObserver *iface) TRACE("(%p) ref=%ld\n", This, ref); - if(!ref) { + if(!ref) release_dispex(&This->dispex); - IDispatch_Release(This->callback); - free(This); - } return ref; } @@ -1213,13 +1210,35 @@ static const IWineMSHTMLMutationObserverVtbl WineMSHTMLMutationObserverVtbl = { MutationObserver_takeRecords }; +static inline struct mutation_observer *mutation_observer_from_DispatchEx(DispatchEx *iface) +{ + return CONTAINING_RECORD(iface, struct mutation_observer, dispex); +} + +static void mutation_observer_unlink(DispatchEx *dispex) +{ + struct mutation_observer *This = mutation_observer_from_DispatchEx(dispex); + unlink_ref(&This->callback); +} + +static void mutation_observer_destructor(DispatchEx *dispex) +{ + struct mutation_observer *This = mutation_observer_from_DispatchEx(dispex); + free(This); +} + +static const dispex_static_data_vtbl_t mutation_observer_dispex_vtbl = { + mutation_observer_destructor, + mutation_observer_unlink +}; + static const tid_t mutation_observer_iface_tids[] = { IWineMSHTMLMutationObserver_tid, 0 }; static dispex_static_data_t mutation_observer_dispex = { L"MutationObserver", - NULL, + &mutation_observer_dispex_vtbl, IWineMSHTMLMutationObserver_tid, mutation_observer_iface_tids };

1 0

Gabriel Ivăncescu : mshtml: Use unlink and destructor in the vtbl for XMLHttpRequest.
by Alexandre Julliard 08 Aug '23

08 Aug '23

Module: wine Branch: master Commit: 7913019fd0baadd921f8a88c3114a7ac7e3c7234 URL: https://gitlab.winehq.org/wine/wine/-/commit/7913019fd0baadd921f8a88c3114a7… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Fri Aug 4 17:30:14 2023 +0300 mshtml: Use unlink and destructor in the vtbl for XMLHttpRequest. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- dlls/mshtml/xmlhttprequest.c | 43 ++++++++++++++++++++++++++++++++----------- 1 file changed, 32 insertions(+), 11 deletions(-) diff --git a/dlls/mshtml/xmlhttprequest.c b/dlls/mshtml/xmlhttprequest.c index 73eea9d7f6d..75f52fb6c8e 100644 --- a/dlls/mshtml/xmlhttprequest.c +++ b/dlls/mshtml/xmlhttprequest.c @@ -550,17 +550,8 @@ static ULONG WINAPI HTMLXMLHttpRequest_Release(IHTMLXMLHttpRequest *iface) TRACE("(%p) ref=%ld\n", This, ref); - if(!ref) { - remove_target_tasks(This->task_magic); - detach_xhr_event_listener(This->event_listener); - if(This->pending_progress_event) - IDOMEvent_Release(&This->pending_progress_event->IDOMEvent_iface); - IHTMLWindow2_Release(&This->window->base.IHTMLWindow2_iface); - release_event_target(&This->event_target); + if(!ref) release_dispex(&This->event_target.dispex); - nsIXMLHttpRequest_Release(This->nsxhr); - free(This); - } return ref; } @@ -1530,6 +1521,35 @@ static inline HTMLXMLHttpRequest *impl_from_DispatchEx(DispatchEx *iface) return CONTAINING_RECORD(iface, HTMLXMLHttpRequest, event_target.dispex); } +static void HTMLXMLHttpRequest_unlink(DispatchEx *dispex) +{ + HTMLXMLHttpRequest *This = impl_from_DispatchEx(dispex); + remove_target_tasks(This->task_magic); + if(This->event_listener) { + XMLHttpReqEventListener *event_listener = This->event_listener; + This->event_listener = NULL; + detach_xhr_event_listener(event_listener); + } + if(This->window) { + HTMLInnerWindow *window = This->window; + This->window = NULL; + IHTMLWindow2_Release(&window->base.IHTMLWindow2_iface); + } + if(This->pending_progress_event) { + DOMEvent *pending_progress_event = This->pending_progress_event; + This->pending_progress_event = NULL; + IDOMEvent_Release(&pending_progress_event->IDOMEvent_iface); + } + unlink_ref(&This->nsxhr); + release_event_target(&This->event_target); +} + +static void HTMLXMLHttpRequest_destructor(DispatchEx *dispex) +{ + HTMLXMLHttpRequest *This = impl_from_DispatchEx(dispex); + free(This); +} + static nsISupports *HTMLXMLHttpRequest_get_gecko_target(DispatchEx *dispex) { HTMLXMLHttpRequest *This = impl_from_DispatchEx(dispex); @@ -1574,7 +1594,8 @@ static void HTMLXMLHttpRequest_init_dispex_info(dispex_data_t *info, compat_mode static event_target_vtbl_t HTMLXMLHttpRequest_event_target_vtbl = { { - NULL, + HTMLXMLHttpRequest_destructor, + HTMLXMLHttpRequest_unlink }, HTMLXMLHttpRequest_get_gecko_target, HTMLXMLHttpRequest_bind_event

1 0

Gabriel Ivăncescu : mshtml: Use unlink and destructor in the vtbl for HTMLRect.
by Alexandre Julliard 08 Aug '23

08 Aug '23

Module: wine Branch: master Commit: adefacfd12bc619266940575a81760087cc41aa2 URL: https://gitlab.winehq.org/wine/wine/-/commit/adefacfd12bc619266940575a81760… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Fri Aug 4 17:30:14 2023 +0300 mshtml: Use unlink and destructor in the vtbl for HTMLRect. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- dlls/mshtml/htmlelem.c | 30 ++++++++++++++++++++++++------ 1 file changed, 24 insertions(+), 6 deletions(-) diff --git a/dlls/mshtml/htmlelem.c b/dlls/mshtml/htmlelem.c index f9ab44ce5fc..653bdd1e6b3 100644 --- a/dlls/mshtml/htmlelem.c +++ b/dlls/mshtml/htmlelem.c @@ -582,12 +582,8 @@ static ULONG WINAPI HTMLRect_Release(IHTMLRect *iface) TRACE("(%p) ref=%ld\n", This, ref); - if(!ref) { - if(This->nsrect) - nsIDOMClientRect_Release(This->nsrect); + if(!ref) release_dispex(&This->dispex); - free(This); - } return ref; } @@ -841,19 +837,41 @@ static const IHTMLRect2Vtbl HTMLRect2Vtbl = { HTMLRect2_get_height, }; +static inline HTMLRect *HTMLRect_from_DispatchEx(DispatchEx *iface) +{ + return CONTAINING_RECORD(iface, HTMLRect, dispex); +} + +static void HTMLRect_unlink(DispatchEx *dispex) +{ + HTMLRect *This = HTMLRect_from_DispatchEx(dispex); + unlink_ref(&This->nsrect); +} + +static void HTMLRect_destructor(DispatchEx *dispex) +{ + HTMLRect *This = HTMLRect_from_DispatchEx(dispex); + free(This); +} + void HTMLRect_init_dispex_info(dispex_data_t *info, compat_mode_t mode) { if (mode >= COMPAT_MODE_IE9) dispex_info_add_interface(info, IHTMLRect2_tid, NULL); } +static const dispex_static_data_vtbl_t HTMLRect_dispex_vtbl = { + HTMLRect_destructor, + HTMLRect_unlink +}; + static const tid_t HTMLRect_iface_tids[] = { IHTMLRect_tid, 0 }; static dispex_static_data_t HTMLRect_dispex = { L"ClientRect", - NULL, + &HTMLRect_dispex_vtbl, IHTMLRect_tid, HTMLRect_iface_tids, HTMLRect_init_dispex_info

1 0

Gabriel Ivăncescu : mshtml: Use unlink and destructor in the vtbl for HTMLRectCollection.
by Alexandre Julliard 08 Aug '23

08 Aug '23

Module: wine Branch: master Commit: ab784433bcf9844cbba85af59c1a4351fdacd949 URL: https://gitlab.winehq.org/wine/wine/-/commit/ab784433bcf9844cbba85af59c1a43… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Fri Aug 4 17:30:14 2023 +0300 mshtml: Use unlink and destructor in the vtbl for HTMLRectCollection. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- dlls/mshtml/htmlelem.c | 22 +++++++++++++++------- 1 file changed, 15 insertions(+), 7 deletions(-) diff --git a/dlls/mshtml/htmlelem.c b/dlls/mshtml/htmlelem.c index 653bdd1e6b3..112057030d3 100644 --- a/dlls/mshtml/htmlelem.c +++ b/dlls/mshtml/htmlelem.c @@ -1084,12 +1084,8 @@ static ULONG WINAPI HTMLRectCollection_Release(IHTMLRectCollection *iface) TRACE("(%p) ref=%ld\n", This, ref); - if(!ref) { - if(This->rect_list) - nsIDOMClientRectList_Release(This->rect_list); + if(!ref) release_dispex(&This->dispex); - free(This); - } return ref; } @@ -1212,6 +1208,18 @@ static inline HTMLRectCollection *HTMLRectCollection_from_DispatchEx(DispatchEx return CONTAINING_RECORD(iface, HTMLRectCollection, dispex); } +static void HTMLRectCollection_unlink(DispatchEx *dispex) +{ + HTMLRectCollection *This = HTMLRectCollection_from_DispatchEx(dispex); + unlink_ref(&This->rect_list); +} + +static void HTMLRectCollection_destructor(DispatchEx *dispex) +{ + HTMLRectCollection *This = HTMLRectCollection_from_DispatchEx(dispex); + free(This); +} + static HRESULT HTMLRectCollection_get_dispid(DispatchEx *dispex, BSTR name, DWORD flags, DISPID *dispid) { HTMLRectCollection *This = HTMLRectCollection_from_DispatchEx(dispex); @@ -1287,8 +1295,8 @@ static HRESULT HTMLRectCollection_invoke(DispatchEx *dispex, DISPID id, LCID lci } static const dispex_static_data_vtbl_t HTMLRectCollection_dispex_vtbl = { - NULL, - NULL, + HTMLRectCollection_destructor, + HTMLRectCollection_unlink, NULL, HTMLRectCollection_get_dispid, HTMLRectCollection_get_name,

1 0