matroskadec: export cover art correctly
authorwm4 <nfxjfg@googlemail.com>
Fri, 3 Apr 2015 14:11:53 +0000 (16:11 +0200)
committerAnton Khirnov <anton@khirnov.net>
Wed, 8 Apr 2015 10:36:53 +0000 (12:36 +0200)
commitc4d37cd9ef6e374bb888f6273259b10fac5bd909
tree415341da9af879ce2952eb5f1dc6ff85f19fbbe5
parent27f274628234c1f934b9a6a6380ed567c1b4ceae
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>
libavformat/matroska.c
libavformat/matroska.h
libavformat/matroskadec.c
libavformat/matroskaenc.c