LGTM.

However,

On Sat, Oct 8, 2016 at 12:20 PM, Rostislav Pehlivanov
<atomnu...@gmail.com> wrote:
> +/**
> + * linear congruential pseudorandom number generator
> + *
> + * @param   previous_val    pointer to the current state of the generator
> + *
> + * @return  Returns a 32-bit pseudorandom integer
> + */
> +static av_always_inline int lcg_random(unsigned previous_val)
> +{
> +    union { unsigned u; int s; } v = { previous_val * 1664525u + 1013904223 
> };
> +    return v.s;
> +}

This could use a comment on where the parameters come from.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to