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

[Alexandre Julliard]

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

Author: Michael Stefaniuc 
Date:   Wed Jun 29 09:55:24 2016 +0200

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

Signed-off-by: Michael Stefaniuc 
Signed-off-by: Alexandre Julliard

---

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;