From 5326f4c7d9554ae2ce0749193d7ed22ea3d45f86 Mon Sep 17 00:00:00 2001 From: Amine Khaldi Date: Fri, 26 Apr 2013 23:05:51 +0000 Subject: [PATCH] [SHELL32] * Fix the FVM_AUTO handing in CDefView::SetCurrentViewMode. Brought to you by Victor Martinez. CORE-7126 #resolve svn path=/trunk/; revision=58863 --- reactos/dll/win32/shell32/shlview.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/reactos/dll/win32/shell32/shlview.cpp b/reactos/dll/win32/shell32/shlview.cpp index bdbfe17ef27..af2d778f5c0 100644 --- a/reactos/dll/win32/shell32/shlview.cpp +++ b/reactos/dll/win32/shell32/shlview.cpp @@ -2407,7 +2407,8 @@ HRESULT STDMETHODCALLTYPE CDefView::SetCurrentViewMode(UINT ViewMode) DWORD dwStyle; TRACE("(%p)->(%u), stub\n", this, ViewMode); - if ((ViewMode < FVM_FIRST || ViewMode > FVM_LAST) /* && (ViewMode != FVM_AUTO) */ ) + /* It's not redundant to check FVM_AUTO because it's a (UINT)-1 */ + if ((ViewMode < FVM_FIRST || ViewMode > FVM_LAST) && (ViewMode != FVM_AUTO)) return E_INVALIDARG; /* Windows before Vista uses LVM_SETVIEW and possibly