Mesa (master): i965: Turn if (query->bo) into an assertion.

Kenneth Graunke kwg at kemper.freedesktop.org
Sat Mar 2 09:19:09 UTC 2013


Module: Mesa
Branch: master
Commit: 614944b8975ce9827b26b92f42ad8b48493eb7f0
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=614944b8975ce9827b26b92f42ad8b48493eb7f0

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Mon Feb 25 23:31:10 2013 -0800

i965: Turn if (query->bo) into an assertion.

The code a few lines above calls brw_emit_query_begin() if !query->bo,
and that creates query->bo.  So it should always be non-NULL.

Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>

---

 src/mesa/drivers/dri/i965/brw_queryobj.c |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_queryobj.c b/src/mesa/drivers/dri/i965/brw_queryobj.c
index 0881ab9..0cdfa6e 100644
--- a/src/mesa/drivers/dri/i965/brw_queryobj.c
+++ b/src/mesa/drivers/dri/i965/brw_queryobj.c
@@ -404,12 +404,12 @@ brw_end_query(struct gl_context *ctx, struct gl_query_object *q)
          brw_emit_query_begin(brw);
       }
 
-      if (query->bo) {
-	 brw_emit_query_end(brw);
+      assert(query->bo);
 
-	 drm_intel_bo_unreference(brw->query.bo);
-	 brw->query.bo = NULL;
-      }
+      brw_emit_query_end(brw);
+
+      drm_intel_bo_unreference(brw->query.bo);
+      brw->query.bo = NULL;
 
       brw->query.obj = NULL;
 




More information about the mesa-commit mailing list