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