[FFmpeg-devel] [PATCH] Use hierarchic names convention (prefix them with av_expr) for eval API.

Michael Niedermayer michaelni
Thu Nov 4 22:20:31 CET 2010


On Thu, Nov 04, 2010 at 10:02:20PM +0100, Stefano Sabatini wrote:
> On date Thursday 2010-11-04 00:20:25 +0100, Michael Niedermayer encoded:
> > On Wed, Nov 03, 2010 at 08:59:33PM +0100, Stefano Sabatini wrote:
> > > More grep-friendly and more consistent with the rest of the FFmpeg
> > > API.
> > > ---
> > >  libavutil/avutil.h |    8 ++++
> > >  libavutil/eval.c   |   96 ++++++++++++++++++++++++++++++++++-----------------
> > >  libavutil/eval.h   |   45 +++++++++++++++++++++---
> > >  3 files changed, 111 insertions(+), 38 deletions(-)
> > 
> > probably ok, maybe wait a day or 2 before applying in case someone has better
> > ideas for the names
> 
> BTW I'm a bit unconfortable with the "and" in
> av_expr_parse_and_eval(), maybe av_expr_parse_eval() should be just
> fine (and shorter).
> 
> But I have no strong feeling on this so opinions and other bikeshed
> color proposals are welcome.

iam slightly in favor of keeping that and in there

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

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101104/bfc02afb/attachment.pgp>



More information about the ffmpeg-devel mailing list