[FFmpeg-devel] [PATCH] eval: add support for a round function

[Stefano Sabatini]

On date Tuesday 2011-04-05 01:00:07 +0200, Michael Niedermayer encoded:
> On Mon, Apr 04, 2011 at 11:36:12PM +0200, Stefano Sabatini wrote:
> > On date Wednesday 2011-03-30 23:49:03 +0200, Michael Niedermayer encoded:
> > > On Wed, Mar 30, 2011 at 10:42:05PM +0200, Stefano Sabatini wrote:
> > > > ---
> > > >  doc/eval.texi    |    9 +++++++++
> > > >  libavutil/eval.c |   23 +++++++++++++++++++++--
> > > >  2 files changed, 30 insertions(+), 2 deletions(-)
> > > > 
> > > > diff --git a/doc/eval.texi b/doc/eval.texi
> > > > index 99cd034..b29e61d 100644
> > > > --- a/doc/eval.texi
> > > > +++ b/doc/eval.texi
> > > > @@ -60,6 +60,15 @@ The function returns the loaded value.
> > > >  Evaluate expression @var{expr} while the expression @var{cond} is
> > > >  non-zero, and returns the value of the last @var{expr} evaluation, or
> > > >  NAN if @var{cond} was always false.
> > > > +
> > > > + at item round(expr1, expr2)
> > > > +Round the value of expression @var{expr1} to a multiple of
> > > > + at var{expr2}.
> > > 
> > > i think round() should be C round() or we will all become very
> > > confused
> > 
> > Updated, round() dropped in favor of ceil/trunc/floor.
> > -- 
> > FFmpeg = Fabulous and Fiendish MultiPurpose Elastic Gymnast
> 
> >  doc/eval.texi    |   12 ++++++++++++
> 
> no comment, human language is not my thing unless its about insults
> 
> 
> >  libavutil/eval.c |   21 +++++++++++++++++++--
> 
> LGTM

Applied.
-- 
FFmpeg = Funny and Funny Mastodontic Puristic Ephemeral Gigant