Patch is Reviewed-by: Boyuan Zhang <boyuan.zh...@amd.com> Regards, Boyuan
-----Original Message----- From: Kai Wasserbäch [mailto:k...@dev.carbon-project.org] Sent: August-20-16 12:15 PM To: mesa-dev@lists.freedesktop.org Cc: Zhang, Boyuan Subject: [PATCH 1/2] st/va: Remove else case in vlVaEndPicture() made superfluous by c59628d11b Commit c59628d11b134fc016388a170880f7646e100d6f made the else statement and duplication of the context->decoder->end_frame() call superfluous. Cc: Boyuan Zhang <boyuan.zh...@amd.com> Signed-off-by: Kai Wasserbäch <k...@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 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
