[FFmpeg-devel] [PATCH] Per-frame metadata
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
I do not really see the bonus between
if (av_size_mult(siezof(foo, n, &size) < 0)
r = av_size_mult(siezof(foo, n, &size);
if (r < 0)
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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel