http://bugs.winehq.org/show_bug.cgi?id=28423
Aric Stewart aric@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |aric@codeweavers.com
--- Comment #15 from Aric Stewart aric@codeweavers.com 2013-09-17 13:12:34 CDT --- (In reply to comment #14)
Created attachment 40573 [details] Patch to fix the issue in wine 1.5.6
The attached patch will fix the problem but as I could not test it fully in windows I'm not submitting it yet. None of the machines in testbot returns valid IMEs to test (tests are skipped).
Ok I am testing on Japanese windows XP.
ret = ImmGetDescriptionW(hkl, NULL, 100); -> Causes and exception ret = ImmGetDescriptionA(hkl, NULL, 100); -> Causes an exception
As for Vista and Windows 7, ImmGetDescription is broken there. It just does not function.
http://blogs.msdn.com/b/michkap/archive/2008/12/04/9173920.aspx
I can try installing a custom old style IME and see but the above windows XP behavior seems to imply to me that crashing in the case of HKL, NULL, 100 is correct.
Back to windows XP;
ret = ImmGetDescriptionW(NULL, NULL, 100); ok(!ret, "ImmGetDescriptionW failed, expected 0 received %d.\n", ret);
That works without issue.
that that implys that your + if (!hKL) return 0;
is correct, but that + if (!lpszDescription) return 0;
is incorrect.
If there is any other testing I can do please ask.