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

[Ronald S. Bultje]

Hi,

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()
anyway...

Ronald