Quoting Mark Thompson (2017-01-05 01:09:12)
> ---
> There was a previous version of this; the logic of this one is the same but 
> slightly cleaner because of the preceding patch.
> 
>  libavcodec/vaapi_encode.c | 18 ++++++++++++++----
>  libavcodec/vaapi_encode.h |  1 +
>  2 files changed, 15 insertions(+), 4 deletions(-)
> 
> diff --git a/libavcodec/vaapi_encode.c b/libavcodec/vaapi_encode.c
> index c670ae62d..246b76abc 100644
> --- a/libavcodec/vaapi_encode.c
> +++ b/libavcodec/vaapi_encode.c
> @@ -605,6 +605,7 @@ static int vaapi_encode_step(AVCodecContext *avctx,
>                  if (err < 0)
>                      return err;
>                  activity = 1;
> +                break;

This could use more explanation.

Otherwise looks ok.

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to