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
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
