comctl32: Remove an always true conditional expression.

Michael Stefaniuc mstefani at redhat.de
Fri Jul 29 05:23:19 CDT 2016


Signed-off-by: Michael Stefaniuc <mstefani at redhat.de>
---
prc is already known to not be NULL thus CopyRect() will always return
TRUE.


 dlls/comctl32/imagelist.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/dlls/comctl32/imagelist.c b/dlls/comctl32/imagelist.c
index 58a7b67..9811da3 100644
--- a/dlls/comctl32/imagelist.c
+++ b/dlls/comctl32/imagelist.c
@@ -3469,7 +3469,9 @@ static HRESULT WINAPI ImageListImpl_GetImageRect(IImageList2 *iface, int i,
     if (!ImageList_GetImageInfo(imgl, i, &info))
         return E_FAIL;
 
-    return CopyRect(prc, &info.rcImage) ? S_OK : E_FAIL;
+    *prc = info.rcImage;
+
+    return S_OK;
 }
 
 static HRESULT WINAPI ImageListImpl_GetIconSize(IImageList2 *iface, int *cx,
-- 
2.7.4



More information about the wine-patches mailing list