[Mesa-dev] [PATCH 1/2] st/va: Remove else case in vlVaEndPicture() made superfluous by c59628d11b

Kai Wasserbäch kai at dev.carbon-project.org
Sat Aug 20 16:14:53 UTC 2016


Commit c59628d11b134fc016388a170880f7646e100d6f made the else statement
and duplication of the context->decoder->end_frame() call superfluous.

Cc: Boyuan Zhang <boyuan.zhang at amd.com>
Signed-off-by: Kai Wasserbäch <kai at dev.carbon-project.org>
---
 src/gallium/state_trackers/va/picture.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/gallium/state_trackers/va/picture.c b/src/gallium/state_trackers/va/picture.c
index 87567be..bbb5595 100644
--- a/src/gallium/state_trackers/va/picture.c
+++ b/src/gallium/state_trackers/va/picture.c
@@ -576,11 +576,9 @@ vlVaEndPicture(VADriverContextP ctx, VAContextID context_id)
       surf->frame_num_cnt = context->desc.h264enc.frame_num_cnt;
       surf->feedback = feedback;
       surf->coded_buf = coded_buf;
-      context->decoder->end_frame(context->decoder, context->target, &context->desc.base);
    }
-   else
-      context->decoder->end_frame(context->decoder, context->target, &context->desc.base);
 
+   context->decoder->end_frame(context->decoder, context->target, &context->desc.base);
    pipe_mutex_unlock(drv->mutex);
    return VA_STATUS_SUCCESS;
 }
-- 
2.8.1



More information about the mesa-dev mailing list