Re: [FFmpeg-devel] [PATCH 1/2] aacenc: use the decoder's lcg PRNG

2016-10-11 Thread Claudio Freire
LGTM. However, On Sat, Oct 8, 2016 at 12:20 PM, Rostislav Pehlivanov wrote: > +/** > + * linear congruential pseudorandom number generator > + * > + * @param previous_valpointer to the current state of the generator > + * > + * @return Returns a 32-bit pseudorandom integer > + */ > +stati

Re: [FFmpeg-devel] [PATCH 1/2] aacenc: use the decoder's lcg PRNG

2016-10-08 Thread Michael Niedermayer
On Sat, Oct 08, 2016 at 04:20:30PM +0100, Rostislav Pehlivanov wrote: > Using lfg was an overkill in this case where the random numbers > were only used for encoder descisions. Should increase result > uniformity between different FPUs and gives a slight speedup. > > Signed-off-by: Rostislav Pehli