[FFmpeg-devel] [PATCH 1/2] libvpx: allow setting 0 for min quantizer

Werner Robitza werner.robitza at gmail.com
Mon Feb 25 18:58:51 CET 2013


> please dont add code in one patch that then gets fixed in the next
>
> also each issue should be dealt with in a seperate patch
> 1 patch to fix the qmin bug
> 1 patch to add bounds checking
>

Apologies. My first patch I ever contributed anywhere.


> also qmin >= qmax should not be silently ignored
>

In that case, qmin won't be changed and is set to the default (4), but a
warning would be appropriate.

I will send patches to deal with each issue separately.

Werner


More information about the ffmpeg-devel mailing list