On Wed, Jan 6, 2010 at 3:28 PM, Detlef Riekenberg <wine....@web.de> 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. -- James Hawkins