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

Reply via email to