On 05/25/2011 06:23 PM, Eric Anholt wrote:
This is a series to fix our Firefox WebGL crashes.  Their debug code,
which they do nightly testing on, uses a debug mode which calls
glFinish() after every operation.  That revealed failures in our
glFinish() implementation.

I'm not really pleased with the state handling for
_ColorDrawBuffers[].  Non-refcounted pointers to refcounted objects
fill me with rage (after filling my bugzilla with crashes).  This is
sort of the minimal patch, while I think a more invasive patch would
be to remove _ColorDrawBuffers[] and just give people an accessor
function using _ColorDrawIndexes[] which are the indexes into
Attachments[] that are updated immediately upon glDrawBuffers().

Reviewed-by: Brian Paul <bri...@vmware.com>

Using an accessor function would be fine with me. I think I put in the _ColorDrawBuffers[] array mostly for swrast.

-Brian
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to