From 1f90d517494021970576d826df64e5b6625361b8 Mon Sep 17 00:00:00 2001 From: "Juan A. Suarez Romero" Date: Tue, 30 Mar 2021 11:00:01 +0200 Subject: [PATCH] v3dv: fix unused value Do not assign to a variable that won't be used. Fixes CID#1468098 "Unused value (UNUSED_VALUE)". Reviewed-by: Iago Toral Quiroga Signed-off-by: Juan A. Suarez Romero Part-of: --- src/broadcom/vulkan/v3dv_cmd_buffer.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/broadcom/vulkan/v3dv_cmd_buffer.c b/src/broadcom/vulkan/v3dv_cmd_buffer.c index 1b5c93927dd..4b5b7b552f9 100644 --- a/src/broadcom/vulkan/v3dv_cmd_buffer.c +++ b/src/broadcom/vulkan/v3dv_cmd_buffer.c @@ -4314,10 +4314,9 @@ cmd_buffer_emit_pre_draw(struct v3dv_cmd_buffer *cmd_buffer) * an active job. In that case, create a new job continuing the current * subpass. */ - struct v3dv_job *job = cmd_buffer->state.job; - if (!job) { - job = v3dv_cmd_buffer_subpass_resume(cmd_buffer, - cmd_buffer->state.subpass_idx); + if (!cmd_buffer->state.job) { + v3dv_cmd_buffer_subpass_resume(cmd_buffer, + cmd_buffer->state.subpass_idx); } /* Restart single sample job for MSAA pipeline if needed */ @@ -4326,7 +4325,7 @@ cmd_buffer_emit_pre_draw(struct v3dv_cmd_buffer *cmd_buffer) /* If the job is configured to flush on every draw call we need to create * a new job now. */ - job = cmd_buffer_pre_draw_split_job(cmd_buffer); + struct v3dv_job *job = cmd_buffer_pre_draw_split_job(cmd_buffer); job->draw_count++; /* GL shader state binds shaders, uniform and vertex attribute state. The