On 07.09.2021 03:26, lance.lmw...@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.
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".
