[PATCH v2 0/1] MR5254: msvcrt: Handle wide specifiers option in __stdio_common_vfscanf()

[Aida Jonikienė (＠DodoGTA)]

This prevents log spam in one game.

--
  v2: msvcrt: Handle wide specifiers option in __stdio_common_vfscanf().

https://gitlab.winehq.org/wine/wine/-/merge_requests/5254

[PATCH v2 1/1] msvcrt: Handle wide specifiers option in __stdio_common_vfscanf().

[Aida Jonikienė]

From: Aida Jonikienė

This prevents log spam in one game.
---
 dlls/msvcrt/scanf.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/msvcrt/scanf.c b/dlls/msvcrt/scanf.c
index 89575101471..c8bbe15f373 100644
--- a/dlls/msvcrt/scanf.c
+++ b/dlls/msvcrt/scanf.c
@@ -710,7 +710,7 @@ int CDECL __stdio_common_vfscanf(unsigned __int64 options,
                                        _locale_t locale,
                                        va_list valist)
 {
-    if (options & ~_CRT_INTERNAL_SCANF_SECURECRT)
+    if (options & ~(_CRT_INTERNAL_SCANF_SECURECRT | _CRT_INTERNAL_SCANF_LEGACY_WIDE_SPECIFIERS))
         FIXME("options %#I64x not handled\n", options);
     if (options & _CRT_INTERNAL_SCANF_SECURECRT)
         return vfscanf_s_l(file, format, locale, valist);

[Aida Jonikienė (＠DodoGTA)]

On Thu Mar  7 21:02:42 2024 +0000, Piotr Caban wrote:
> The flag has only effect on wide scanf and can be ignored in __stdio_common_vfscanf.
> Please change the code to:
>         if (options & ~(_CRT_INTERNAL_SCANF_SECURECRT | _CRT_INTERNAL_SCANF_LEGACY_WIDE_SPECIFIERS))
>             FIXME("options %#I64x not handled\n", options);

I just updated the MR to handle this

[PATCH v2 0/1] MR5254: msvcrt: Handle wide specifiers option in __stdio_common_vfscanf() - approved

[Piotr Caban (＠piotr)]

This merge request was approved by Piotr Caban.