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

Michael Niedermayer michaelni at gmx.at
Tue Apr 5 01:00:07 CEST 2011


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

[..]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- 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/20110405/07cb6167/attachment.asc>


More information about the ffmpeg-devel mailing list