[PATCH 7/9] quartz/videorenderer: Use assignment instead of CopyRect

Gabriel Ivăncescu gabrielopcode at gmail.com
Fri Dec 28 06:23:03 CST 2018


Signed-off-by: Gabriel Ivăncescu <gabrielopcode at gmail.com>
---
 dlls/quartz/videorenderer.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/dlls/quartz/videorenderer.c b/dlls/quartz/videorenderer.c
index 61f08f5..1d2db3e 100644
--- a/dlls/quartz/videorenderer.c
+++ b/dlls/quartz/videorenderer.c
@@ -483,7 +483,7 @@ static const BaseWindowFuncTable renderer_BaseWindowFuncTable = {
 static HRESULT WINAPI VideoRenderer_GetSourceRect(BaseControlVideo* iface, RECT *pSourceRect)
 {
     VideoRendererImpl *This = impl_from_BaseControlVideo(iface);
-    CopyRect(pSourceRect,&This->SourceRect);
+    if (pSourceRect) *pSourceRect = This->SourceRect;
     return S_OK;
 }
 
@@ -549,7 +549,7 @@ static HRESULT WINAPI VideoRenderer_GetStaticImage(BaseControlVideo* iface, LONG
 static HRESULT WINAPI VideoRenderer_GetTargetRect(BaseControlVideo* iface, RECT *pTargetRect)
 {
     VideoRendererImpl *This = impl_from_BaseControlVideo(iface);
-    CopyRect(pTargetRect,&This->DestRect);
+    if (pTargetRect) *pTargetRect = This->DestRect;
     return S_OK;
 }
 
@@ -616,14 +616,14 @@ static HRESULT WINAPI VideoRenderer_SetDefaultTargetRect(BaseControlVideo* iface
 static HRESULT WINAPI VideoRenderer_SetSourceRect(BaseControlVideo* iface, RECT *pSourceRect)
 {
     VideoRendererImpl *This = impl_from_BaseControlVideo(iface);
-    CopyRect(&This->SourceRect,pSourceRect);
+    if (pSourceRect) This->SourceRect = *pSourceRect;
     return S_OK;
 }
 
 static HRESULT WINAPI VideoRenderer_SetTargetRect(BaseControlVideo* iface, RECT *pTargetRect)
 {
     VideoRendererImpl *This = impl_from_BaseControlVideo(iface);
-    CopyRect(&This->DestRect,pTargetRect);
+    if (pTargetRect) This->DestRect = *pTargetRect;
     return S_OK;
 }
 
-- 
2.19.1




More information about the wine-devel mailing list