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

Reply via email to