Mesa (master): mesa: tidy up _mesa_DeleteVertexArrays()

Samuel Pitoiset hakzsam at kemper.freedesktop.org
Thu Jul 20 14:16:33 UTC 2017


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

Author: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Date:   Thu Jul 20 11:52:58 2017 +0200

mesa: tidy up _mesa_DeleteVertexArrays()

Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>

---

 src/mesa/main/arrayobj.c | 25 ++++++++++++-------------
 1 file changed, 12 insertions(+), 13 deletions(-)

diff --git a/src/mesa/main/arrayobj.c b/src/mesa/main/arrayobj.c
index 17a789f4da..ce0050ace3 100644
--- a/src/mesa/main/arrayobj.c
+++ b/src/mesa/main/arrayobj.c
@@ -493,19 +493,18 @@ _mesa_DeleteVertexArrays(GLsizei n, const GLuint *ids)
    for (i = 0; i < n; i++) {
       struct gl_vertex_array_object *obj = _mesa_lookup_vao(ctx, ids[i]);
 
-      if ( obj != NULL ) {
-	 assert( obj->Name == ids[i] );
-
-	 /* If the array object is currently bound, the spec says "the binding
-	  * for that object reverts to zero and the default vertex array
-	  * becomes current."
-	  */
-	 if ( obj == ctx->Array.VAO ) {
-	    _mesa_BindVertexArray(0);
-	 }
-
-	 /* The ID is immediately freed for re-use */
-	 remove_array_object(ctx, obj);
+      if (obj) {
+         assert(obj->Name == ids[i]);
+
+         /* If the array object is currently bound, the spec says "the binding
+          * for that object reverts to zero and the default vertex array
+          * becomes current."
+          */
+         if (obj == ctx->Array.VAO)
+            _mesa_BindVertexArray(0);
+
+         /* The ID is immediately freed for re-use */
+         remove_array_object(ctx, obj);
 
          if (ctx->Array.LastLookedUpVAO == obj)
             _mesa_reference_vao(ctx, &ctx->Array.LastLookedUpVAO, NULL);




More information about the mesa-commit mailing list