From: Vasyl Vavrychuk <vasyl.vavrychuk@opensynergy.com>
Date: Sun, 16 Jan 2022 07:33:14 +0000 (+0100)
Subject: Rearrange buffer valid and busy check.
X-Git-Tag: 12.92.0~2
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?p=apps%2Fcamera-gstreamer.git;a=commitdiff_plain;h=5573029323801248c5ba7f88df6e594ec4b79af0;hp=d6adb1a32912c65752b6163f2914faa0d42a3e74

Rearrange buffer valid and busy check.

It is more natural to check buffer validity first before buffer
busyness.

Fixes https://gerrit.automotivelinux.org/gerrit/c/apps/camera-gstreamer/+/26935/2/app/main.cpp#161

Bug-AGL: SPEC-4148
Signed-off-by: Vasyl Vavrychuk <vasyl.vavrychuk@opensynergy.com>
Change-Id: I0df3dd48c183429b55f49915e6722342a97bd2c5
---

diff --git a/app/main.cpp b/app/main.cpp
index d84ca26..da80fd3 100644
--- a/app/main.cpp
+++ b/app/main.cpp
@@ -158,7 +158,7 @@ get_next_buffer(struct window *window)
 	 * the 'old' one and force creation of the buffer with the newer
 	 * dimensions */
 	if (window->wait_for_configure && window->maximized) {
-		if (!window->buffers[0].busy && window->buffers[0].buffer) {
+		if (window->buffers[0].buffer && !window->buffers[0].busy) {
 			wl_buffer_destroy(window->buffers[0].buffer);
 			window->buffers[0].buffer = NULL;
 			window->wait_for_configure = false;