[FFmpeg-devel] [PATCH] gxf: add timecode information to metadata

[Reimar Döffinger]

On Mon, Oct 31, 2011 at 08:04:05PM +0100, Matthieu Bouron wrote:
> Patch updated,
> Does it look good to you ?

Almost.
I mean putting the full 64 bit TRACK_AUX into the gxf_stream_info
and then checking the track type and adding metadata after coming
back from the tag parsing function (ok, whether the full 64 bit
or just the time code does not really matter, just seems nicer going
with the full 64 bit).
I.e. the tag parsing function really should only parse the tags
into the gxf_stream_info, processing them should be in the function
one level up.