Re: wine/dlls/x11drv x11drv_main.c
It crashed every time when I closed an app, not just "sometimes" :( And I think maybe exchange the calling order of XCloseIM and XCloseDisplay is a better method than just comments out XCloseIM ( I think this will lead to some other problems). --- Alexandre Julliard <julliard(a)wine.codeweavers.com> wrote:
ChangeSet ID: 7168 CVSROOT: /opt/cvs-commit Module name: wine Changes by: julliard(a)wine.codeweavers.com 2003/01/29 19:08:53
Modified files: dlls/x11drv : x11drv_main.c
Log message: XCloseIM sometimes crashes in Xlib, don't call it.
Patch: http://cvs.winehq.com/patch.py?id=7168
Old revision New revision Changes Path 1.67 1.68 +1 -1 wine/dlls/x11drv/x11drv_main.c
__________________________________________________ Do you Yahoo!? Yahoo! Mail Plus - Powerful. Affordable. Sign up now. http://mailplus.yahoo.com
liu spider <liuspider(a)yahoo.com> writes:
And I think maybe exchange the calling order of XCloseIM and XCloseDisplay is a better method than just comments out XCloseIM ( I think this will lead to some other problems).
Exchanging them makes it crash on my machine, so I think we have to avoid the close, unless someone wants to chase the bug. I suspect the problem is in Xlib though, we're probably the only app trying to properly close an IM... -- Alexandre Julliard julliard(a)winehq.com
hi Alexandre Could you tell me the progress of the work on adding XIM support to WINE? I just found that I could not even trigger my XIM using the lates CVS. Thanks
Exchanging them makes it crash on my machine, so I think we have to avoid the close, unless someone wants to chase the bug. I suspect the problem is in Xlib though, we're probably the only app trying to properly close an IM...
-- Alexandre Julliard julliard(a)winehq.com
__________________________________________________ Do you Yahoo!? Yahoo! Tax Center - forms, calculators, tips, more http://taxes.yahoo.com/
liu spider <liuspider(a)yahoo.com> writes:
hi Alexandre Could you tell me the progress of the work on adding XIM support to WINE?
Not much progress lately I'm afraid, I've had other priorities.
I just found that I could not even trigger my XIM using the lates CVS.
Is that a new bug or was it always this way? -- Alexandre Julliard julliard(a)winehq.com
--- Alexandre Julliard <julliard(a)winehq.com> wrote:
Is that a new bug or was it always this way?
Oh, sorry, I just found it's the XIM's fault , not related to wine.
Not much progress lately I'm afraid, I've had other priorities.
I use the "old method" to implement XIM support using the newly added functions in wine. It works in some cases, but don't in other. And should I submit the patch or just wait for you to work on it later? __________________________________________________ Do you Yahoo!? Yahoo! Tax Center - forms, calculators, tips, more http://taxes.yahoo.com/
liu spider <liuspider(a)yahoo.com> writes:
I use the "old method" to implement XIM support using the newly added functions in wine. It works in some cases, but don't in other. And should I submit the patch or just wait for you to work on it later?
It never hurts to submit a patch. Even if I don't apply it as is, it can still be a useful reference for further work. -- Alexandre Julliard julliard(a)winehq.com
participants (2)
-
Alexandre Julliard -
liu spider