On Fri, 2004-01-23 at 15:48, Roland Scheidegger wrote:
> Ian Romanick wrote:
> > Roland Scheidegger wrote:
> >
> >> - if ( ctx->Color.ColorLogicOpEnabled ) {
> >> + if ( ctx->Color.ColorLogicOpEnabled || (ctx->Color.BlendEnabled &&
> >> mode == GL_LOGIC_OP)) {
> >
> >
> > There's derrived state that Mesa calculates just for this case. The
> > test should be 'if ( ctx->Color._LogicOpEnabled )'.
>
> Yes, saw that derived state. For some reason I can't figure out now, I
> thought it might not be a good idea to use Mesa derived state directly.
> Ah well, this patch corrects this (even simpler, and should also save 1
> or 2 clock cycles ;-)).
Committed, thanks.
--
Earthling Michel DÃnzer | Debian (powerpc), X and DRI developer
Libre software enthusiast | http://svcs.affero.net/rm.php?r=daenzer
-------------------------------------------------------
The SF.Net email is sponsored by EclipseCon 2004
Premiere Conference on Open Tools Development and Integration
See the breadth of Eclipse activity. February 3-5 in Anaheim, CA.
http://www.eclipsecon.org/osdn
--
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel