[PATCH 1/3] x11drv: Move cached cursor position from under x11's
lock and into it's own.
Vitaliy Margolen
wine-devel at kievinfo.com
Sun Oct 15 23:48:42 CDT 2006
Mike McCormack wrote:
>
> Vitaliy Margolen wrote:
>
>> POINT cursor_pos;
>>
>> +static CRITICAL_SECTION cursor_CritSection;
>
> There's a reference to cursor_pos in dlls/winex11.drv/keyboard.c too,
> and you're missing a critical section around it.
>
Oh thank you. I think it will need a temp variable there. It's inside
server call which is a critical section too. And trying to enter two
crit_sections is not a good idea AFAIK.
Vitaliy
More information about the wine-devel
mailing list