Quoting Mark Thompson (2016-09-18 20:34:35)
> This allows better checking of capabilities and will make it easier
> to add more functionality later.
> 
> It also commonises some duplicated code around rate control setup
> and adds more comments explaining the internals.
> ---
>  libavcodec/vaapi_encode.c       | 253 
> ++++++++++++++++++++++++++--------------
>  libavcodec/vaapi_encode.h       |  96 ++++++++++-----
>  libavcodec/vaapi_encode_h264.c  | 253 
> +++++++++++++++-------------------------
>  libavcodec/vaapi_encode_h265.c  | 208 ++++++++++++---------------------
>  libavcodec/vaapi_encode_mjpeg.c |  45 +++----
>  5 files changed, 418 insertions(+), 437 deletions(-)

Looks generally ok. I don't feel qualified to judge whether all the
reorders are fine, but I assume you know what you're doing there :)

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to