Module: Mesa Branch: master Commit: 1c9db39d54508608ea9518bc82eacbd8e27c410c URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=1c9db39d54508608ea9518bc82eacbd8e27c410c
Author: Matt Turner <matts...@gmail.com> Date: Sat Apr 11 10:05:31 2015 -0700 i965: Don't bother freeing NULL. Commit e16c5c90 was replacing 'region' with 'mt', leaving this nonsensical code. Reviewed-by: Jason Ekstrand <jason.ekstr...@intel.com> --- src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c index c0a3452..0424a87 100644 --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c @@ -749,10 +749,8 @@ intel_miptree_create_for_bo(struct brw_context *brw, width, height, depth, true, 0, false, disable_aux_buffers); - if (!mt) { - free(mt); - return mt; - } + if (!mt) + return NULL; drm_intel_bo_reference(bo); mt->bo = bo; _______________________________________________ mesa-commit mailing list mesa-commit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-commit