[PATCH v2 13/15] mshtml: Use unlink and destructor in the vtbl for HTMLOptionElementFactory.

[Gabriel Ivăncescu]

From: Gabriel Ivăncescu

Signed-off-by: Gabriel Ivăncescu 
---
 dlls/mshtml/htmlselect.c | 18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)

diff --git a/dlls/mshtml/htmlselect.c b/dlls/mshtml/htmlselect.c
index cbbd9ebcc69..3558539b322 100644
--- a/dlls/mshtml/htmlselect.c
+++ b/dlls/mshtml/htmlselect.c
@@ -487,10 +487,8 @@ static ULONG WINAPI HTMLOptionElementFactory_Release(IHTMLOptionElementFactory *

TRACE("(%p) ref=%ld\n", This, ref);

-    if(!ref) {
+    if(!ref)
         release_dispex(&This->dispex);
-        free(This);
-    }

return ref;
 }
@@ -591,6 +589,16 @@ static inline HTMLOptionElementFactory *HTMLOptionElementFactory_from_DispatchEx
     return CONTAINING_RECORD(iface, HTMLOptionElementFactory, dispex);
 }

+static void HTMLOptionElementFactory_unlink(DispatchEx *dispex)
+{
+}
+
+static void HTMLOptionElementFactory_destructor(DispatchEx *dispex)
+{
+    HTMLOptionElementFactory *This = HTMLOptionElementFactory_from_DispatchEx(dispex);
+    free(This);
+}
+
 static HRESULT HTMLOptionElementFactory_value(DispatchEx *dispex, LCID lcid,
         WORD flags, DISPPARAMS *params, VARIANT *res, EXCEPINFO *ei,
         IServiceProvider *caller)
@@ -629,8 +637,8 @@ static const tid_t HTMLOptionElementFactory_iface_tids[] = {
 };

static const dispex_static_data_vtbl_t HTMLOptionElementFactory_dispex_vtbl = {
-    NULL,
-    NULL,
+    HTMLOptionElementFactory_destructor,
+    HTMLOptionElementFactory_unlink,
     HTMLOptionElementFactory_value,
     NULL,
     NULL,