[FFmpeg-devel] [PATCH 4/4] avcodec/nvenc: Add constrainedFrame encoding support

[Timo Rothenpieler]

On 07.09.2021 03:26, lance.lmwang at gmail.com wrote:
> On Mon, Sep 06, 2021 at 02:49:47PM +0200, Timo Rothenpieler wrote:
>> Series applied.
>>
>> While testing, I found that constrained encoding for HEVC was not a thing
>> until SDK 10.0, so I added the necessary check and compile time guards for
>> that.
>>
>> Also done some other minor style and logic changes, and renamed the options
>> to use "-" instead of "_", so the options match their libx264 counterpart.
> Thanks, I notice libx264 use "-" style, but it's not consistent with other options
> of nvenc. So I use "_" instead. I'm not sure what's style is more preferable.
> 

The style of nvenc options is already all over the place, and staying in 
line with libx264 seems a bit more important here.

Maybe at some future major bump the options can be cleaned up a bit.
-------------- 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/20210907/8c41625d/attachment.bin>