On Thu, Jan 28, 2016 at 3:52 PM, Michael Niedermayer
<mich...@niedermayer.cc> wrote:
> 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
>
The entire file doesn't use contexts for some reason, I would rather
keep it like this in this patch and then send a follow up to make all
the log commands in that file use contexts.
- Hendrik
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
