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

Nicolas George nicolas.george at normalesup.org
Tue Apr 12 19:01:08 CEST 2011


Le tridi 23 germinal, an CCXIX, Stefano Sabatini a écrit :
> Nit: multiply

Fixed locally.

> -1 -> AVERROR(EINVAL)

That require mem.h to include error.h, and this is the only conceivable
error for av_size_mult, so I am mildly in favor of keeping -1. But if you
insist on AVERROR(EINVAL) I will change.

> Would it make sense to move metadata to libavutil rather than to
> libavcodec?
> 
> For example you may need to display metadata from a lavfi filter.

libavutil was my first choice, as it has absorbed libavcore; but Michael
seemed to prefer libavcodec.

If you think this is better in libavutil I will make the change.

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/20110412/591a97ed/attachment.asc>


More information about the ffmpeg-devel mailing list