Michael Stefaniuc : strmbase: Use SetRect() instead of open coding it.

Alexandre Julliard julliard at winehq.org
Sun Jul 3 12:18:27 CDT 2016


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

Author: Michael Stefaniuc <mstefani at redhat.de>
Date:   Wed Jun 29 09:55:24 2016 +0200

strmbase: Use SetRect() instead of open coding it.

Signed-off-by: Michael Stefaniuc <mstefani at redhat.de>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/strmbase/video.c | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/dlls/strmbase/video.c b/dlls/strmbase/video.c
index 9714d7e..ec3a076 100644
--- a/dlls/strmbase/video.c
+++ b/dlls/strmbase/video.c
@@ -368,10 +368,8 @@ HRESULT WINAPI BaseControlVideoImpl_SetSourcePosition(IBasicVideo *iface, LONG L
     BaseControlVideo *This = impl_from_IBasicVideo(iface);
 
     TRACE("(%p/%p)->(%d, %d, %d, %d)\n", This, iface, Left, Top, Width, Height);
-    SourceRect.left = Left;
-    SourceRect.top = Top;
-    SourceRect.right = Left + Width;
-    SourceRect.bottom = Top + Height;
+
+    SetRect(&SourceRect, Left, Top, Left + Width, Top + Height);
     This->pFuncsTable->pfnSetSourceRect(This, &SourceRect);
 
     return S_OK;
@@ -408,10 +406,7 @@ HRESULT WINAPI BaseControlVideoImpl_SetDestinationPosition(IBasicVideo *iface, L
 
     TRACE("(%p/%p)->(%d, %d, %d, %d)\n", This, iface, Left, Top, Width, Height);
 
-    DestRect.left = Left;
-    DestRect.top = Top;
-    DestRect.right = Left + Width;
-    DestRect.bottom = Top + Height;
+    SetRect(&DestRect, Left, Top, Left + Width, Top + Height);
     This->pFuncsTable->pfnSetTargetRect(This, &DestRect);
 
     return S_OK;




More information about the wine-cvs mailing list