loader/dri3: Simplify buffer age query
authorRoman Gilg <subdiff@gmail.com>
Mon, 24 Jan 2022 16:14:44 +0000 (17:14 +0100)
committerMarge Bot <emma+marge@anholt.net>
Tue, 25 Oct 2022 08:30:35 +0000 (08:30 +0000)
The ternary operator is hard to read. Instead initialize the return variable
to zero and adjust its value in an if-clause.

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14684>

src/loader/loader_dri3_helper.c

index 81245f4..7219c4f 100644 (file)
@@ -1209,11 +1209,11 @@ int
 loader_dri3_query_buffer_age(struct loader_dri3_drawable *draw)
 {
    struct loader_dri3_buffer *back = dri3_find_back_alloc(draw);
-   int ret;
+   int ret = 0;
 
    mtx_lock(&draw->mtx);
-   ret = (!back || back->last_swap == 0) ? 0 :
-      draw->send_sbc - back->last_swap + 1;
+   if (back && back->last_swap != 0)
+      ret = draw->send_sbc - back->last_swap + 1;
    mtx_unlock(&draw->mtx);
 
    return ret;