[FFmpeg-devel] [PATCH 2/2] Make ff_parse_expr() and ff_parse_and_eval_expr() return an int containing an error code.

Michael Niedermayer michaelni
Thu May 20 19:30:31 CEST 2010


On Thu, May 20, 2010 at 09:21:03AM +0200, Stefano Sabatini wrote:
> On date Thursday 2010-05-20 03:07:08 +0200, Michael Niedermayer encoded:
> > On Thu, May 20, 2010 at 02:08:46AM +0200, Stefano Sabatini wrote:
> > > Allow these functions to convey to the calling application the reason
> > > of the failure, which is not always due to a parsing error but for
> > > example it may depend on a memory problem.
> > 
> > this makes the code more complex
> > so iam not sure if this is a good idea.
> > Could you explain what use you see in the error code?
> 
> I'm not very strong about this change, currently the only practical
> use I can see is for distinguishing an invalid input error from a
> memory error, as they are semantically two very different things, so
> I'd prefer to allow programs to distinguish amongst the two.

then maybe pass a int *error and alow it to be NULL
its imho more convenient, but its drifting toward bikeshed


-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100520/cf8d7e18/attachment.pgp>



More information about the ffmpeg-devel mailing list