[FFmpeg-devel] [PATCH] Per-frame metadata

Nicolas George nicolas.george at normalesup.org
Wed Apr 13 13:07:44 CEST 2011


Le tridi 23 germinal, an CCXIX, Stefano Sabatini a écrit :
> In many cases we propagate the error code, not having to remap it is a
> bonus.

I do not really see the bonus between

	if (av_size_mult(siezof(foo, n, &size) < 0)
	    return AVERROR(EINVAL);

and

	r = av_size_mult(siezof(foo, n, &size);
	if (r < 0)
	    return r;

The only benefit I see is in term of API uniformity, thus the question:

Do you think this is worth making mem.h depend on error.h?

> My argument is that metadata is not codec-specific, so I consider
> libavutil a better place (and at least per-frame metadata filtering
> looks senseful, even if I can't imagine other uses outside libavcodec
> at the moment).

I perfectly understand the argument, and in fact I agree with it. I will
wait a bit more to see if there are other remarks, especially from Michael.

Regards,

-- 
  Nicolas George
-------------- 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/20110413/60cf4458/attachment.asc>


More information about the ffmpeg-devel mailing list