Re: [AD] XIM patch for Allegro 4.1.x

[Elias Pschernig <elias@xxxxxxxxxx>]

On Tue, 2004-10-26 at 16:26 +1000, Peter Wang wrote:
> >
> >The patch now does two things. First, it replaces X locking with
> >pthreads locking. Second, it adds a new keyboard driver, which works the
> >same as the one in new_api_branch - just it has support for XIM. 
> >
> 
> Cool!  I will test it out some more later, but I saw a problem in 
> exkeys: modifier keys should not stuff characters into the readkey() buffer.
> 

Hm, you're right. I'll look into it.

> >I also had to fix a small bug in xdga2.c where the mode fetching called
> >X11 functions without proper locking.
> >
> 
> You can probably commit this part right now.
> 

Will do.


> We need a solution for this.  I think the vsync-emulation could run off 
> the bgman thread instead of the timer thread.  This will also deal with 
> the problem of the vsync-emulation not working if the timer subsystem is 
> not installed.
> 

It's interesting that this doesn't happen with current CVS. Maybe it is
connected to now using the MMX for color conversion. I don't know which
of the examples triggered it.. I was going through all of them to test
the XIM patch, and one example triggered it. I'll put back vsync and see
which one it is.

-- 
Elias Pschernig