[FFmpeg-devel] [PATCH] doc/eval: fix/extend documentation for root() function

[Michael Niedermayer]

On Wed, Jan 30, 2013 at 11:55:41PM +0100, Stefano Sabatini wrote:
> On date Wednesday 2013-01-30 00:00:53 +0100, Michael Niedermayer encoded:
> > On Sat, Jan 26, 2013 at 11:16:28AM +0100, Stefano Sabatini wrote:
> [...]
> > >  @item root(expr, max)
> > > -Finds x where f(x)=0 in the interval 0..max.
> > > -f() must be continuous or the result is undefined.
> > > +Find a value where a function represented by @var{expr} with argument
> > > + at var{ld(0)} is 0 in the interval 0.. at var{max}.
> > > +
> > > +The expression in @var{expr} must denote a continuous function or the
> > > +result is undefined.
> > > +
> > 
> > > + at var{ld(0)} is used to represent the function value, that means that
> > 
> > this is not correct
> > you mix value and argument, maybe you should use "input" and "output"
> > instead
> > 
> > 
> > > +the given expression will be evaluated multiple times with various
> > > +values of x that the expression can access through @code{ld(0)}. When
> > > +the expression evaluates to 0 then the value of @code{ld(0)} will be
> > > +returned.
> > >  @end table
> 
> Up.
> -- 
> FFmpeg = Free and Forgiving Mere Perennial Ermetic Geek

>  eval.texi |   13 +++++++++++--
>  1 file changed, 11 insertions(+), 2 deletions(-)
> 0d4704bd82dd2c347ed09b1265dd170e5d29c68a  0001-doc-eval-fix-extend-documentation-for-root-function.patch
> From 57f5eeefc1c49a9cb1a7788d2cb9df0cb4f286a0 Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefasab at gmail.com>
> Date: Wed, 23 Jan 2013 11:45:31 +0100
> Subject: [PATCH] doc/eval: fix/extend documentation for root() function

LGTM

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

During times of universal deceit, telling the truth becomes a
revolutionary act. -- George Orwell
-------------- 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/20130131/a4dda4f3/attachment.asc>