[FFmpeg-devel] [PATCH] WIP/eval: document recently added random(), hypot(), gcd().

[Michael Niedermayer]

On Mon, Oct 10, 2011 at 09:47:30PM +0200, Clément Bœsch wrote:
> On Sat, Oct 08, 2011 at 02:35:25PM +0200, Michael Niedermayer wrote:
> > On Sat, Oct 08, 2011 at 12:26:34PM +0200, Clément Bœsch wrote:
> > > On Sat, Oct 08, 2011 at 12:25:00PM +0200, Clément Bœsch wrote:
> > > > ---
> > > > I took av_gcd() description to document gcd(), but I'm not sure for random()
> > > > (it seems normalized but I didn't check). And for hypot(), I just don't know
> > > > what it is… :)
> > 
> > random is a PRNG with value between 0.0 and 1.0 using the argument as
> > index into our array of variables and the indexed to value as
> > seed/state
> > and array of variables in the sense of ld() st()
> > 
> > hypot() is just the same as the C function
> > stolen from the fine manual:
> >  -- Function: double hypot (double X, double Y)
> >  -- Function: float hypotf (float X, float Y)
> >  -- Function: long double hypotl (long double X, long double Y)
> >      These functions return `sqrt (X*X + Y*Y)'.  This is the length of
> >      the hypotenuse of a right triangle with sides of length X and Y,
> >      or the distance of the point (X, Y) from the origin.  Using this
> >      function instead of the direct formula is wise, since the error is
> >      much smaller.  See also the function `cabs' in *note Absolute
> >      Value::.
> 
> Thanks, then what about the attached patch?

LGTM

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111010/57b33072/attachment.asc>