[FFmpeg-devel] [PATCH 9/9] Make avfilter_graph_parse() always return meaningful error codes.
Michael Niedermayer
michaelni
Sun Nov 7 16:03:19 CET 2010
On Sun, Nov 07, 2010 at 03:51:23PM +0100, Stefano Sabatini wrote:
> On date Friday 2010-11-05 18:20:33 +0100, Stefano Sabatini encoded:
> > ---
> > libavfilter/graphparser.c | 22 ++++++++++++----------
> > 1 files changed, 12 insertions(+), 10 deletions(-)
>
> I'll apply the whole in three days if there are no objections.
I have no objections to this patchset except that the error codes need to be
checked.
simply passing error codes from a function to the outter is not guranteed to
be correct.
for ENOMEM surely it should work out most of the time, but a EINVAL for a
function is not neccesarily a EINVAL for the outer function
if you checked these things then ok
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- 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/20101107/514aed55/attachment.pgp>
More information about the ffmpeg-devel
mailing list