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?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2628#note_29584