From 662b72e2f65026001fbe39ffd8f0c2f9fdd0adfc Mon Sep 17 00:00:00 2001 From: Moses Turner <moses@collabora.com> Date: Fri, 18 Mar 2022 19:18:10 -0500 Subject: [PATCH] u/sink: Tiny refactor for queue --- src/xrt/auxiliary/util/u_sink_queue.c | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/src/xrt/auxiliary/util/u_sink_queue.c b/src/xrt/auxiliary/util/u_sink_queue.c index 6f672368c..7ab45a8fc 100644 --- a/src/xrt/auxiliary/util/u_sink_queue.c +++ b/src/xrt/auxiliary/util/u_sink_queue.c @@ -98,20 +98,20 @@ queue_pop(struct u_sink_queue *q) static bool queue_try_refpush(struct u_sink_queue *q, struct xrt_frame *xf) { - if (!queue_is_full(q)) { - struct u_sink_queue_elem *elem = U_TYPED_CALLOC(struct u_sink_queue_elem); - xrt_frame_reference(&elem->frame, xf); - elem->next = NULL; - if (q->back == NULL) { // First frame - q->front = elem; - } else { // Next frame - q->back->next = elem; - } - q->back = elem; - q->size++; - return true; + if (queue_is_full(q)) { + return false; } - return false; + struct u_sink_queue_elem *elem = U_TYPED_CALLOC(struct u_sink_queue_elem); + xrt_frame_reference(&elem->frame, xf); + elem->next = NULL; + if (q->back == NULL) { // First frame + q->front = elem; + } else { // Next frame + q->back->next = elem; + } + q->back = elem; + q->size++; + return true; } //! Clears the queue and unreferences all of its frames.