[FFmpeg-devel] [PATCH] metadata compatibility layer (part2)

Ronald S. Bultje rsbultje
Thu Jan 8 00:28:58 CET 2009


On Tue, Jan 6, 2009 at 7:21 PM, Aurelien Jacobs <aurel at gnuage.org> wrote:
> It is slightly ugly, especially the AVFormatContext pointer stored
> in the AVMetadata struct, but that's the cleanest way I found.

Why do you do it in av_metadata_set()? Why not in av_write_header(),
where you actually have access to the AVFormatContext? That way, this
ugly API isn't needed, and _all_ practical cases that you want to
support set these metadata fields before calling av_write_header()


More information about the ffmpeg-devel mailing list