[FFmpeg-devel] [PATCH] lavc/vaapi_encode_h264: disable B frame in baseline profile

Mark Thompson sw at jkqxz.net
Sun Jan 8 01:20:37 EET 2017


On 06/01/17 15:19, Moritz Barsnick wrote:
> Since Michael mentioned it:
> 
> On Fri, Dec 16, 2016 at 10:21:25 +0800, Jun Zhao wrote:
> 
>> +        if (avctx->max_b_frames != 0) {
>> +            avctx->max_b_frames = 0;
>> +            av_log(avctx, AV_LOG_WARNING, "H.264 constrained baseline "
>> +                   "profile don't support encode B frame.\n");
>> +        }
> 
> "H.264 constrained baseline profile doesn't support encoding with B
> frames, disabling them.\n".
> 
>> +        if (avctx->max_b_frames != 0) {
>> +            avctx->max_b_frames = 0;
>> +            av_log(avctx, AV_LOG_WARNING, "H.264 baseline "
>> +                   "profile don't support encode B frame.\n");
>> +        }
> 
> "H.264 baseline profile doesn't support encoding with B frames,
> disabling them.\n".

+1 to this rephrasing (for improved grammar and clarity).  Patch otherwise fine.

Also apologies for missing the mail last month, thank you for the ping.

- Mark


More information about the ffmpeg-devel mailing list