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