From fbd9ed0dee968142227e0b0a320da0f81eee0ca1 Mon Sep 17 00:00:00 2001 From: Rodrigo Rivas Costa Date: Thu, 16 Feb 2017 21:59:39 +0100 Subject: [PATCH] user32: Fix error handling in EndDeferWindowPos. Signed-off-by: Sebastian Lackner Signed-off-by: Alexandre Julliard --- dlls/user32/tests/win.c | 1 - dlls/user32/winpos.c | 9 ++++----- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/dlls/user32/tests/win.c b/dlls/user32/tests/win.c index 0046c7cf512..70c13705bbd 100644 --- a/dlls/user32/tests/win.c +++ b/dlls/user32/tests/win.c @@ -9539,7 +9539,6 @@ todo_wine DestroyWindow(hwnd); ret = EndDeferWindowPos(hdwp); -todo_wine ok(ret, "got %d\n", ret); } diff --git a/dlls/user32/winpos.c b/dlls/user32/winpos.c index 2fa194fa1c5..d94f5a3fb8e 100644 --- a/dlls/user32/winpos.c +++ b/dlls/user32/winpos.c @@ -2430,7 +2430,6 @@ BOOL WINAPI EndDeferWindowPos( HDWP hdwp ) { DWP *pDWP; WINDOWPOS *winpos; - BOOL res = TRUE; int i; TRACE("%p\n", hdwp); @@ -2442,20 +2441,20 @@ BOOL WINAPI EndDeferWindowPos( HDWP hdwp ) return FALSE; } - for (i = 0, winpos = pDWP->winPos; res && i < pDWP->actualCount; i++, winpos++) + for (i = 0, winpos = pDWP->winPos; i < pDWP->actualCount; i++, winpos++) { TRACE("hwnd %p, after %p, %d,%d (%dx%d), flags %08x\n", winpos->hwnd, winpos->hwndInsertAfter, winpos->x, winpos->y, winpos->cx, winpos->cy, winpos->flags); if (WIN_IsCurrentThread( winpos->hwnd )) - res = USER_SetWindowPos( winpos ); + USER_SetWindowPos( winpos ); else - res = SendMessageW( winpos->hwnd, WM_WINE_SETWINDOWPOS, 0, (LPARAM)winpos ); + SendMessageW( winpos->hwnd, WM_WINE_SETWINDOWPOS, 0, (LPARAM)winpos ); } HeapFree( GetProcessHeap(), 0, pDWP->winPos ); HeapFree( GetProcessHeap(), 0, pDWP ); - return res; + return TRUE; } -- 2.11.4.GIT