[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