Alexandre Julliard : msdmo: Don' t use an interface pointer for an internal function.

Alexandre Julliard julliard at winehq.org
Mon Nov 29 13:38:41 CST 2010


Module: wine
Branch: master
Commit: e59ac81c5e2a0acab249379b5c2c524fe87e5836
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=e59ac81c5e2a0acab249379b5c2c524fe87e5836

Author: Alexandre Julliard <julliard at winehq.org>
Date:   Mon Nov 29 12:25:53 2010 +0100

msdmo: Don't use an interface pointer for an internal function.

---

 dlls/msdmo/dmoreg.c |   10 ++++------
 1 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/dlls/msdmo/dmoreg.c b/dlls/msdmo/dmoreg.c
index 2c55296..0d5d397 100644
--- a/dlls/msdmo/dmoreg.c
+++ b/dlls/msdmo/dmoreg.c
@@ -333,12 +333,10 @@ lend:
 
 
 /**************************************************************************
-*   IEnumDMO_Destructor
+*   IEnumDMOImpl_Destructor
 */
-static BOOL IEnumDMO_Destructor(IEnumDMO* iface)
+static BOOL IEnumDMOImpl_Destructor(IEnumDMOImpl* This)
 {
-    IEnumDMOImpl *This = impl_from_IEnumDMO(iface);
-
     TRACE("%p\n", This);
 
     if (This->hkey)
@@ -418,7 +416,7 @@ static IEnumDMO * IEnumDMO_Constructor(
 lerr:
         if(!ret)
         {
-            IEnumDMO_Destructor((IEnumDMO*)lpedmo);
+            IEnumDMOImpl_Destructor(lpedmo);
             HeapFree(GetProcessHeap(),0,lpedmo);
             lpedmo = NULL;
         }
@@ -477,7 +475,7 @@ static ULONG WINAPI IEnumDMO_fnRelease(IEnumDMO * iface)
 
     if (!refCount)
     {
-        IEnumDMO_Destructor((IEnumDMO*)This);
+        IEnumDMOImpl_Destructor(This);
         HeapFree(GetProcessHeap(),0,This);
     }
     return refCount;




More information about the wine-cvs mailing list