[FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

Timo Rothenpieler timo at rothenpieler.org
Tue Feb 28 17:23:28 EET 2017


> Patch looks good to me too. Timo, I can push it if you don't mind not
> testing it yourself.

I have some stilistic nits, otherwise it's ok.
It leaves the fall through comment dangling, even though there is no
fall through anymore.
And it could use fallthrough to in the new location of RC_(2_PASS_)VBR,
but that's clearly optional and I have no strong opinion on what's better.


> 
> --phil
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 


More information about the ffmpeg-devel mailing list