Re: [PATCH 0/6] MR2628: Some more shell32 improvements.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/shell32/tests/shlview.c:
>      }
>
>      hr = IFolderView_SetCurrentViewMode(fview, 9);
> -
>    ok(hr == E_INVALIDARG || broken(hr == S_OK),
> -
>       "got (0x%08lx)\n", hr);
> -
>    ok(hr == S_OK || broken(hr == E_INVALIDARG) /* pre win7 */, "Got hr %#lx.\n", hr);
> -
>    ok(viewmode == FVM_CONTENT || broken(viewmode == FVM_THUMBSTRIP) /* vista */, "Got view mode %d.\n", viewmode);

Is this missing a call to GetCurrentViewMode()? So the conclusion here is that any larger number gives last valid mode?