diff --git a/win32ss/user/ntuser/defwnd.c b/win32ss/user/ntuser/defwnd.c index c8c25164f92..09b05485209 100644 --- a/win32ss/user/ntuser/defwnd.c +++ b/win32ss/user/ntuser/defwnd.c @@ -401,20 +401,19 @@ UserPaintCaption(PWND pWnd, INT Flags) { if (pWnd->state & WNDS_HASCAPTION && pWnd->head.pti->MessageQueue == gpqForeground) Flags |= DC_ACTIVE; - /* + /* * When themes are not enabled we can go on and paint the non client area. * However if we do that with themes enabled we will draw a classic frame. * This is solved by sending a themes specific message to notify the themes - * engine that the caption needs to be redrawn + * engine that the caption needs to be redrawn */ if (gpsi->dwSRVIFlags & SRVINFO_APIHOOK) { - /* + /* * This will cause uxtheme to either paint the themed caption or call * RealUserDrawCaption in order to draw the classic caption when themes * are disabled but the themes service is enabled */ - TRACE("UDCB Flags %08x\n"); co_IntSendMessage(UserHMGetHandle(pWnd), WM_NCUAHDRAWCAPTION, Flags, 0); } else @@ -436,7 +435,7 @@ DefWndSetIcon(PWND pWnd, WPARAM wParam, LPARAM lParam) HICON hIcon, hIconSmall, hIconOld; if ( wParam > ICON_SMALL2 ) - { + { EngSetLastError(ERROR_INVALID_PARAMETER); return 0; } @@ -486,8 +485,7 @@ DefWndGetIcon(PWND pWnd, WPARAM wParam, LPARAM lParam) case ICON_SMALL2: hIconRet = UserGetProp(pWnd, gpsi->atomIconSmProp, TRUE); break; - default: - break; + DEFAULT_UNREACHABLE; } return (LRESULT)hIconRet; } diff --git a/win32ss/user/ntuser/nonclient.c b/win32ss/user/ntuser/nonclient.c index 0ee5a9eeda0..99b212aef2d 100644 --- a/win32ss/user/ntuser/nonclient.c +++ b/win32ss/user/ntuser/nonclient.c @@ -626,7 +626,7 @@ PCURICON_OBJECT FASTCALL NC_IconForWindow( PWND pWnd ) // it does not use the default icon! And it does not check for DS_MODALFRAME. if (!hIcon && !(pWnd->ExStyle & WS_EX_DLGMODALFRAME)) { - if (!hIcon) hIcon = gpsi->hIconSmWindows; // Both are IDI_WINLOGO Small + hIcon = gpsi->hIconSmWindows; // Both are IDI_WINLOGO Small if (!hIcon) hIcon = gpsi->hIconWindows; // Reg size. } if (hIcon) @@ -1181,7 +1181,7 @@ LRESULT NC_HandleNCCalcSize( PWND Wnd, WPARAM wparam, RECTL *Rect, BOOL Suspende SIZE WindowBorders; RECT OrigRect; LONG Style = Wnd->style; - LONG exStyle = Wnd->ExStyle; + LONG exStyle = Wnd->ExStyle; if (Rect == NULL) { @@ -1558,7 +1558,7 @@ NC_HandleNCLButtonDblClk(PWND pWnd, WPARAM wParam, LPARAM lParam) { PMENU SysMenu = IntGetSystemMenu(pWnd, FALSE); UINT state = IntGetMenuState(UserHMGetHandle(SysMenu), SC_CLOSE, MF_BYCOMMAND); - + /* If the close item of the sysmenu is disabled or not present do nothing */ if ((state & (MF_DISABLED | MF_GRAYED)) || (state == 0xFFFFFFFF)) break; @@ -1577,7 +1577,7 @@ NC_HandleNCLButtonDblClk(PWND pWnd, WPARAM wParam, LPARAM lParam) * * Handle a WM_NCRBUTTONDOWN message. Called from DefWindowProc(). */ -LRESULT NC_HandleNCRButtonDown( PWND pwnd, WPARAM wParam, LPARAM lParam ) +LRESULT NC_HandleNCRButtonDown(PWND pwnd, WPARAM wParam, LPARAM lParam) { MSG msg; INT hittest = wParam; diff --git a/win32ss/user/ntuser/sysparams.c b/win32ss/user/ntuser/sysparams.c index 2265219586c..1a61efd55bd 100644 --- a/win32ss/user/ntuser/sysparams.c +++ b/win32ss/user/ntuser/sysparams.c @@ -93,8 +93,6 @@ static const WCHAR* KEY_KDBPREF = L"Control Panel\\Accessibility\\Keyboard Prefe static const WCHAR* KEY_SCRREAD = L"Control Panel\\Accessibility\\Blind Access"; static const WCHAR* VAL_ON = L"On"; - - /** Loading the settings ******************************************************/ static @@ -146,7 +144,7 @@ SpiLoadTimeOut(VOID) { return 0; } - if (wcslen(szApplicationName) == 0) return 0; + if (szApplicationName[0] == 0) return 0; return SpiLoadInt(KEY_DESKTOP, VAL_SCRTO, 0); } @@ -933,7 +931,7 @@ SpiGetSet(UINT uiAction, UINT uiParam, PVOID pvParam, FLONG fl) /* Fixup user's structure size */ metrics->cbSize = sizeof(NONCLIENTMETRICSW); - + if (!SpiSet(&gspv.ncm, metrics, sizeof(NONCLIENTMETRICSW), fl)) return 0;