On Tue, 28 Jul 2015 12:39:59 -0700 Sasi Inguva <is...@google.com> wrote:
> Signed-off-by: Sasi Inguva <is...@google.com> > --- > libavformat/matroska.h | 1 + > libavformat/matroskadec.c | 21 +++++++++++++- > libavformat/matroskaenc.c | 66 > ++++++++++++++++++++++++++++++++++++++------ > tests/fate/wavpack.mak | 4 +-- > tests/ref/acodec/tta | 4 +-- > tests/ref/fate/binsub-mksenc | 2 +- > tests/ref/lavf/mkv | 8 +++--- > tests/ref/seek/lavf-mkv | 44 ++++++++++++++--------------- > 8 files changed, 109 insertions(+), 41 deletions(-) > Am I right with these assumptions: - this will write tags by default, while it didn't before this patch - the tags are written to the end of the stream (and not close to the start of it) If so, this will require a seek to the end of the file when loading it, while this could be avoided before. It breaks fast loading of network streams. I'm not really fond of this. (I know, mkvmerge commits similar sins.) _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel