kernel32: Print a FIXME only once in LCMapStringEx
Andrew Eikum
aeikum at codeweavers.com
Mon Apr 25 10:03:44 CDT 2016
On Mon, Apr 25, 2016 at 04:58:55PM +0200, Louis Lenders wrote:
> diff --git a/dlls/kernel32/locale.c b/dlls/kernel32/locale.c
> index 7418da9..8901870 100644
> --- a/dlls/kernel32/locale.c
> +++ b/dlls/kernel32/locale.c
> @@ -2974,10 +2974,13 @@ INT WINAPI LCMapStringEx(LPCWSTR name, DWORD flags, LPCWSTR src, INT srclen, LPW
> LPNLSVERSIONINFO version, LPVOID reserved, LPARAM lparam)
> {
> LPWSTR dst_ptr;
> + static int once;
>
> if (version) FIXME("unsupported version structure %p\n", version);
> if (reserved) FIXME("unsupported reserved pointer %p\n", reserved);
> - if (lparam) FIXME("unsupported lparam %lx\n", lparam);
> +
> + if (!once++)
> + if (lparam) FIXME("unsupported lparam %lx\n", lparam);
I think you got the if statements backwards here. You should print the
first time lparam is non-zero, not only if it's non-zero on the first
LCMapStringEx call.
Andrew
More information about the wine-devel
mailing list