[PATCH] qedit: Remove redundant "return 0" from Release functions.

Alex Henrie alexhenrie24 at gmail.com
Thu Apr 28 00:12:54 CDT 2016


Cc: Andrew Eikum <aeikum at codeweavers.com>
Cc: Nikolay Sivov <nsivov at codeweavers.com>

Signed-off-by: Alex Henrie <alexhenrie24 at gmail.com>
---
 dlls/qedit/mediadet.c      | 1 -
 dlls/qedit/samplegrabber.c | 2 --
 dlls/qedit/timeline.c      | 3 ---
 3 files changed, 6 deletions(-)

diff --git a/dlls/qedit/mediadet.c b/dlls/qedit/mediadet.c
index 53ab75a..b7a5abb 100644
--- a/dlls/qedit/mediadet.c
+++ b/dlls/qedit/mediadet.c
@@ -112,7 +112,6 @@ static ULONG WINAPI MediaDet_inner_Release(IUnknown *iface)
     {
         MD_cleanup(This);
         CoTaskMemFree(This);
-        return 0;
     }
 
     return ref;
diff --git a/dlls/qedit/samplegrabber.c b/dlls/qedit/samplegrabber.c
index 946dba9..e3f387b 100644
--- a/dlls/qedit/samplegrabber.c
+++ b/dlls/qedit/samplegrabber.c
@@ -92,7 +92,6 @@ static ULONG WINAPI Single_IEnumMediaTypes_Release(IEnumMediaTypes *iface)
         if (This->mtype.pbFormat)
             CoTaskMemFree(This->mtype.pbFormat);
         CoTaskMemFree(This);
-        return 0;
     }
     return refCount;
 }
@@ -335,7 +334,6 @@ static ULONG WINAPI SampleGrabber_Release(IUnknown *iface)
     {
         SampleGrabber_cleanup(This);
         CoTaskMemFree(This);
-        return 0;
     }
     return ref;
 }
diff --git a/dlls/qedit/timeline.c b/dlls/qedit/timeline.c
index 8c6ac5b..450e796 100644
--- a/dlls/qedit/timeline.c
+++ b/dlls/qedit/timeline.c
@@ -103,10 +103,7 @@ static ULONG WINAPI Timeline_Release(IUnknown *iface)
     TRACE("(%p) new ref = %u\n", This, ref);
 
     if (ref == 0)
-    {
         CoTaskMemFree(This);
-        return 0;
-    }
 
     return ref;
 }
-- 
2.8.0




More information about the wine-patches mailing list