Changeset c4d37cd9 in ffmpeg

Timestamp:

Apr 8, 2015, 10:36:53 AM (10 years ago)

Author:

Anton Khirnov <anton@khirnov.net>

Branches:

master

Children:

8e1717cf, bb815dfc

Parents:

27f27462

git-author:

wm4 <nfxjfg@googlemail.com> (04/03/15 14:11:53)

git-committer:

Anton Khirnov <anton@khirnov.net> (04/08/15 10:36:53)

Message:

matroskadec: export cover art correctly

Generally, libavformat exports cover art pictures as video streams with
1 packet and AV_DISPOSITION_ATTACHED_PIC set. Only matroskadec exported
it as attachment with codec_id set to AV_CODEC_ID_MJPEG.

Obviously, this should be consistent, so change the Matroska demuxer to
export a AV_DISPOSITION_ATTACHED_PIC pseudo video stream.

Matroska muxing is probably incorrect too. I know that it can create
broken files with an audio track and just 1 video frame when e.g.
remuxing mp3 with APIC to mkv. But for now this commit does not change
anything about muxing, and also continues to write attachments with
AV_CODEC_ID_MJPEG should the muxer application have special knowledge
that the Matroska is broken in this way.

Signed-off-by: Anton Khirnov <​anton@khirnov.net>

(No files)