Re: mlang: [RESEND] Print a FIXME only for unused parameter (Wine FIXME Report)

[James Hawkins]

On Wed, Jan 6, 2010 at 3:28 PM, Detlef Riekenberg  wrote:
> According to MSDN, both parameter are unused and must be 0/NULL
> A FIXME for every call is useless
>
> (there is no PATCH 1/2, so a resend without that prefix)
>
> --
> By by ... Detlef
>
> dlls/mlang/mlang.c |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/dlls/mlang/mlang.c b/dlls/mlang/mlang.c
> index 49f0605..9abe124 100644
> --- a/dlls/mlang/mlang.c
> +++ b/dlls/mlang/mlang.c
> @@ -3007,9 +3007,6 @@ exit:
>     return hr;
>  }
>
> -/*
> -
> - TODO: handle dwFlag and lpFallBack
>
> -*/
>  static HRESULT WINAPI fnIMultiLanguage2_ConvertStringToUnicodeEx(
>     IMultiLanguage3* iface,
>     DWORD* pdwMode,
> @@ -3021,7 +3018,10 @@ static HRESULT WINAPI fnIMultiLanguage2_ConvertStringToUnicodeEx(
>     DWORD dwFlag,
>     WCHAR* lpFallBack)
>  {
> - FIXME("\n");
> - if (dwFlag || (lpFallBack != NULL))
> - FIXME("Ignoring dwFlag (0x%x/%d) and lpFallBack (%p)\n",
> - dwFlag, dwFlag, lpFallBack);
> -

if (dwFlag || lpFallBack)

seems to be simpler and easier to read.