[Bug 3165] Patch available
Mike McCormack
mike at codeweavers.com
Thu Aug 18 19:39:09 CDT 2005
Duane Clark wrote:
>> Well the patch does improve the cursor. I can now see it. Although it
>> is only grey. It there any reason why the best cursor selection method
>> always checks for bits == 1. What about color cursors?
>>
>
> I don't really understand how the patch fixed things, but...
> Riven uses a bunch of color cursors (hands pointing various directions).
> Without the patch, only the standard pointer cursor was generated. This
> cursor appeared and disappeared normally (the cursor is supposed to
> disappear while some Quicktime clips are playing, and then reappear).
> With the patch, the color cursors reappeared and all are correct.
It should improve things over the original code that was there in
cursoricon.c revision 1.9. That code first checked if there was only
one cursor, then returned it, but if there was more than on cursor, it
would choose one a 1bpp cursor. That scheme would miss out picking a
cursor at all if there was two coloured cursors.
The new scheme makes sure that we always start with one cursor and
improve over that.
I think the problem is that standard X cannot support coloured cursors.
The Xcursor extension appears to, but we don't support that. Any
volunteers?
Mike
More information about the wine-devel
mailing list