Nikolay Sivov : strmbase: Remove a couple of redundant statements in Release() methods.

Alexandre Julliard julliard at winehq.org
Fri Nov 25 14:19:18 CST 2016


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

Author: Nikolay Sivov <nsivov at codeweavers.com>
Date:   Fri Nov 25 12:10:44 2016 +0300

strmbase: Remove a couple of redundant statements in Release() methods.

Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/strmbase/pin.c       | 4 +---
 dlls/strmbase/transform.c | 5 +----
 2 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/dlls/strmbase/pin.c b/dlls/strmbase/pin.c
index d61469c..c19f528 100644
--- a/dlls/strmbase/pin.c
+++ b/dlls/strmbase/pin.c
@@ -397,10 +397,8 @@ ULONG WINAPI BaseOutputPinImpl_Release(IPin * iface)
     TRACE("(%p)->() Release from %d\n", iface, refCount + 1);
 
     if (!refCount)
-    {
         BaseOutputPin_Destroy(This);
-        return 0;
-    }
+
     return refCount;
 }
 
diff --git a/dlls/strmbase/transform.c b/dlls/strmbase/transform.c
index 4ad9730..92eef16 100644
--- a/dlls/strmbase/transform.c
+++ b/dlls/strmbase/transform.c
@@ -344,11 +344,8 @@ ULONG WINAPI TransformFilterImpl_Release(IBaseFilter * iface)
         IUnknown_Release(This->seekthru_unk);
         BaseFilter_Destroy(&This->filter);
         CoTaskMemFree(This);
-
-        return 0;
     }
-    else
-        return refCount;
+    return refCount;
 }
 
 /** IMediaFilter methods **/




More information about the wine-cvs mailing list