Nikolay Sivov (@nsivov) commented about include/shtypes.idl:
> SCALE_450_PERCENT = 450,
> SCALE_500_PERCENT = 500
> } DEVICE_SCALE_FACTOR;
> +
> +typedef [v1_enum] enum tagSFBS_FLAGS {
> + SFBS_FLAGS_ROUND_TO_NEAREST_DISPLAYED_DIGIT = 0x0001,
> + SFBS_FLAGS_TRUNCATE_UNDISPLAYED_DECIMAL_DIGITS = 0x0002
> +} SFBS_FLAGS;
I don't see this in current SDK, it's not defined in an idl.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2609#note_31102
Nikolay Sivov (@nsivov) commented about dlls/shlwapi/string.c:
> + {
> + case SFBS_FLAGS_ROUND_TO_NEAREST_DISPLAYED_DIGIT:
> + dBytes = round(dBytes / bfFormats[i].dDivisor) / bfFormats[i].dNormaliser;
> + break;
> + case SFBS_FLAGS_TRUNCATE_UNDISPLAYED_DECIMAL_DIGITS:
> + dBytes = floor(dBytes / bfFormats[i].dDivisor) / bfFormats[i].dNormaliser;
> + break;
> + default:
> + return E_INVALIDARG;
> + }
>
> + /* Not sure about the error code. Can't find a way to trigger the error here */
> if (!FormatDouble(dBytes, bfFormats[i].nDecimals, lpszDest, cchMax))
> - return NULL;
> + return E_UNEXPECTED;
> +
E_FAIL is probably more generic. Also comment is not really needed here. I'd also remove documentation comments before function declaration, leaving just function name there.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2609#note_31101
--
v2: shell32: Update view mode flags while creating IShellView for IExplorerBrowser.
shell32: Reimplement IExplorerBrowser_SetFolderSettings() with support of setting flags.
shell32/tests: Move tests of IExplorerBrowser_SetFolderSettings() to a separated function and more tests.
shell32: Reimplement IFolderView2_SetCurrentViewMode() with modern behaviours.
shell32: Implement IFolderView2_{Get, Set}CurrentFolderFlags().
shell32/tests: Add tests for IFolderView2_{Get, Set}CurrentFolderFlags().
https://gitlab.winehq.org/wine/wine/-/merge_requests/2628
This is needed for mod.io integration in game "The Entropy Centre". Without support for this option the game hangs when trying to authenticate.
Please note that i do not know how Wine works, and have very little understanding of what i am doing. I copied and adjusted code from `connect_query_option()` in the same file. The game does work with this patch though.
--
v2: winhttp: Avoid unnecessary cast in connect_query_option().
winhttp: Support WINHTTP_OPTION_PARENT_HANDLE in request_query_option().
https://gitlab.winehq.org/wine/wine/-/merge_requests/2697
This is required by https://bugs.winehq.org/show_bug.cgi?id=54660 .
--
v7: vkd3d-shader/hlsl: Consider duplicated input semantic types equivalent in SM1.
vkd3d-shader/hlsl: Handle possibly different types in input semantic var load.
vkd3d-shader/hlsl: Error out when a semantic is used with incompatible types.
vkd3d-shader/hlsl: Error out when an output semantic is used more than once.
vkd3d-shader/hlsl: Support semantics for array types.
vkd3d-shader/hlsl: Don't create semantic vars more than once.
vkd3d-shader/hlsl: Move get_array_size() and get_array_type() to hlsl.c.
tests: Test duplicated semantics.
tests: Test array types with semantics.
vkd3d-shader/hlsl: Avoid invalid input/output copies for non-numeric types.
tests: Map unindentified hrs on compilation.
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/148
--
v2: winepulse: Use dedicated macros to call interface functions.
wineoss: Use dedicated macros to call interface functions.
winecoreaudio: Use dedicated macros to call interface functions.
winealsa: Use dedicated macros to call interface functions.
https://gitlab.winehq.org/wine/wine/-/merge_requests/2695