fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-15 16:28:04 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

intel: Remove gratuitous context checks in intel_delete_renderbuffer().

Browse source 
Even if we don't have a current context, if we're freeing the rb we
should free its region (and BO).  The renderbuffer unreference checks
appear to be just cargo-cult from the region unreference code.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=30217
Reviewed-by: Chad Versace <chad@chad-versace.us>
(cherry picked from commit 007c2d6cd2)
This commit is contained in:
Eric Anholt 2011-06-29 11:37:35 -07:00
parent e3e99be131
commit 55a75856fb
1 changed files with 5 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
src/mesa/drivers/dri/intel/intel_fbo.c
View file

@ -70,24 +70,15 @@ intel_new_framebuffer(struct gl_context * ctx, GLuint name)
static void
intel_delete_renderbuffer(struct gl_renderbuffer *rb)
{
GET_CURRENT_CONTEXT(ctx);
struct intel_context *intel = intel_context(ctx);
struct intel_renderbuffer *irb = intel_renderbuffer(rb);
ASSERT(irb);
if (intel && irb->region) {
intel_region_release(&irb->region);
}
if (intel && irb->hiz_region) {
intel_region_release(&irb->hiz_region);
}
if (intel && irb->wrapped_depth) {
_mesa_reference_renderbuffer(&irb->wrapped_depth, NULL);
}
if (intel && irb->wrapped_stencil) {
_mesa_reference_renderbuffer(&irb->wrapped_stencil, NULL);
}
intel_region_release(&irb->region);
intel_region_release(&irb->hiz_region);
_mesa_reference_renderbuffer(&irb->wrapped_depth, NULL);
_mesa_reference_renderbuffer(&irb->wrapped_stencil, NULL);
free(irb);
}