Hi Paul,

On Sun, Apr 28, 2019 at 16:42:45 +0200, Paul B Mahol wrote:

Cosmetic nits:

> +Alowed range is from 1 to 6.
   ^ Allowed

> +Alowed range is from 2 to 256.
   ^
Ditto.

> +    float *bbufers[256];
              ^
I'm not sure of the intent, but would this be "buffers" or "bbuffers"?
> +static float lut_lookup(const float *lut, const int val1, const int val2)
> +{
> +    return lut[val1 > val2 ? val1 - val2 : val2 - val1];

FFABS() may help for readability.

I can't judge on the rest, as always.

Cheers,
Moritz
_______________________________________________
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".

Reply via email to