2017-11-11 4:57 GMT+01:00 Aman Gupta <ffm...@tmm1.net>:
> - status = CMVideoFormatDescriptionGetH264ParameterSetAtIndex(vid_fmt,
> - 0,
> - NULL,
> - NULL,
> - &ps_count,
> - NULL);
> + status = vtctx->get_param_set_func(vid_fmt,
> + 0,
> + NULL,
> + NULL,
> + &ps_count,
> + NULL);
Please don't do this: It makes reviewing your changes (not only now)
unnecessarily difficult
> - if(vtctx->has_b_frames && vtctx->profile == H264_PROF_BASELINE){
> - av_log(avctx, AV_LOG_WARNING, "Cannot use B-frames with baseline
> profile. Output will not contain B-frames.\n");
> - vtctx->has_b_frames = false;
> + vtctx->has_b_frames = avctx->max_b_frames > 0;
> + if(vtctx->has_b_frames && vtctx->profile == H264_PROF_BASELINE){
> + av_log(avctx, AV_LOG_WARNING, "Cannot use B-frames with baseline
> profile. Output will not contain B-frames.\n");
> + vtctx->has_b_frames = false;
This is much worse: Do not re-indent, push another commit with
white-space changes only.
Thank you, Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
