Re: [FFmpeg-devel] [PATCH 3/3] ffmpeg_dxva2: add a profile check for hevc

[Michael Niedermayer]

On Thu, Jan 28, 2016 at 12:58:27PM +0100, Hendrik Leppkes wrote:
> ---
>  ffmpeg_dxva2.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/ffmpeg_dxva2.c b/ffmpeg_dxva2.c
> index 1b62bb9..35bedec 100644
> --- a/ffmpeg_dxva2.c
> +++ b/ffmpeg_dxva2.c
> @@ -637,6 +637,12 @@ int dxva2_init(AVCodecContext *s)
>          return AVERROR(EINVAL);
>      }
>  
> +    if (s->codec_id == AV_CODEC_ID_HEVC &&
> +        s->profile != FF_PROFILE_HEVC_MAIN && s->profile != 
> FF_PROFILE_HEVC_MAIN_10) {
> +        av_log(NULL, loglevel, "Unsupported HEVC profile for DXVA2 HWAccel: 
> %d\n", s->profile);
                  ^^^^
this should be some context if possible

otherwise LGTM

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato

signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel