On 19.02.2021 16:29, Ricardo Monteiro wrote:
Hello,

The patch fixes a bug regarding the number of B frames being used: The default number of B frames is now -1 for both h264_nvenc and hevc_nvenc. This allows the number of B frames to be determined based on the coding preset or overwritten using “-bf” in CLI.

The condition change seems unnecessary, given that the new one does the exact same logic.

I'm in favor of changing the bf default, even though it technically changes behavior of existing command lines. It seems counter intuitive that the encoder would gimp itself by disabling bframes by default.

If no one objects, I will apply this.

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to