On 24 May 2011 22:37, Stefan Dösinger stefan@codeweavers.com wrote:
I put the fixme checks outside context_check_fbo_status because the ENTER_GL() / LEAVE_GL() locking is more expensive than the FBO validation itself. Moving the locking into the function
I'm not sure that's really worth it. If the GL lock is really such a concern, you should probably change the calls in context.c to already be under the GL lock anyway. You can easily change context_apply_draw_buffers() for example to require locking by the caller.