[FFmpeg-devel] [PATCH] avcodec/nvenc: Fixing default number of B frames

Timo Rothenpieler timo at rothenpieler.org
Fri Feb 19 19:01:42 EET 2021


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.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4494 bytes
Desc: S/MIME Cryptographic Signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20210219/52e592e9/attachment.bin>


More information about the ffmpeg-devel mailing list