Around 17 o'clock on Jul 17, Johannes Rath wrote:

> >             format = PICT_FORMAT (bpp, PICT_TYPE_COLOR, v, 0, 0, 0);
>
> I do not understand in the first place is why v (simply a loop index) is
> used as an parameter for PICT_FORMAT().

The visual index is passed from this line down to the creation of the 
format stored in this value, so it's not (actually) a typo, but it
is a misuse of the macro.  Instead, a different PICT_FORMAT macro
should be created for PICT_TYPE_COLOR/PICT_TYPE_GRAY which can hold
the entire visual index, even if the server has more than 16 visuals.

Keith Packard        XFree86 Core Team        HP Cambridge Research Lab


_______________________________________________
Xpert mailing list
[EMAIL PROTECTED]
http://XFree86.Org/mailman/listinfo/xpert

Reply via email to