Nikolay Sivov (@nsivov) commented about dlls/dwrite/analyzer.c:
return S_OK; }
+static struct fallback_locale * font_fallback_get_locale_exact(const struct list *locales, + const WCHAR *locale_name) +{ + struct fallback_locale *locale; + + LIST_FOR_EACH_ENTRY(locale, locales, struct fallback_locale, entry) + { + if (!wcsicmp(locale->name, locale_name)) return locale; + } + + return NULL; +}
This is fine but we don't need to duplicate. Could you add a bool flag to the font_fallback_get_locale(), either 'use_neutral' or, an opposite to it, 'exact_match' to block second condition. P.S. stdbool.h is already included and used, so "bool" is good for internal things. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10201#note_131070