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