On 24 May 2011 22:37, Stefan Dösinger <ste...@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.