[PATCH libX11 0/2] Mend some XIM design issues.

Egbert Eich eich at freedesktop.org
Thu Jan 3 08:58:43 PST 2013


Happy New Year!

On Fri, Dec 07, 2012 at 05:57:50AM -0500, Egbert Eich wrote:
> These two patches attempt to fix some issues with the design
> of XIM - the X Input Method.
[...]
> Noneteless the patches have been 'field-tested' on SUSE enterprise
> products for quite some time now - while those fixes have eliminated
> the error that has been seen so far we are not aware of any reports 
> of regressions caused by those fixes.


Any takers for these - anyone willing to ack?

I know, XIM is painful and I understand if noone wants to bother:
The underlying issues are really messy.
On the other hand these patches have resolved the bugs described
and no regression has been seen so far although these patches
have been used on SUSE Enterprise products for quite a while now.

XIM the underlying input method doesn't play an important role any
more today, it is still used by a lot of legacy applications and
software that is (or at least used to be) based on legacy X11 
libraries.

The bugs addressed here have been found in Java applications.

Therefore I believe it would still be beneficial to include these 
fixes in XIM.

> 
> Egbert Eich (2):
>   XIM: Fix race on focus change: set 'FABRICATED' only when keyev
>     filters in place.
>   XIM: Fix sync problem on focus change.
> 
>  modules/im/ximcp/imDefIc.c   |   27 +++++++++++++++++++++++++++
>  modules/im/ximcp/imDefLkup.c |    4 +++-
>  2 files changed, 30 insertions(+), 1 deletions(-)
> 
> -- 
> 1.7.7

Cheers,
	Egbert.


More information about the xorg-devel mailing list