mesa: trivial simplifications in _mesa_reference_buffer_object_
authorMarek Olšák <marek.olsak@amd.com>
Sun, 21 Aug 2022 17:59:41 +0000 (13:59 -0400)
committerMarge Bot <emma+marge@anholt.net>
Mon, 26 Sep 2022 22:58:16 +0000 (22:58 +0000)
The else statement conditions were tautological.

Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/18828>

src/mesa/main/bufferobj.c

index 4f528ce..64a4da3 100644 (file)
@@ -1079,25 +1079,22 @@ _mesa_reference_buffer_object_(struct gl_context *ctx,
          if (p_atomic_dec_zero(&oldObj->RefCount)) {
             _mesa_delete_buffer_object(ctx, oldObj);
          }
-      } else if (ctx == oldObj->Ctx) {
+      } else {
          /* Update the private ref count. */
          assert(oldObj->CtxRefCount >= 1);
          oldObj->CtxRefCount--;
       }
-
-      *ptr = NULL;
    }
-   assert(!*ptr);
 
    if (bufObj) {
       /* reference new buffer */
       if (shared_binding || ctx != bufObj->Ctx)
          p_atomic_inc(&bufObj->RefCount);
-      else if (ctx == bufObj->Ctx)
+      else
          bufObj->CtxRefCount++;
-
-      *ptr = bufObj;
    }
+
+   *ptr = bufObj;
 }