[FFmpeg-devel] [PATCH v2] avformat/movenc: properly handle cover image codecs
michael at niedermayer.cc
Sat Jun 2 01:46:30 EEST 2018
On Fri, Jun 01, 2018 at 02:24:39AM +0300, Timo Teräs wrote:
> Find codec tag for attached images using appropriate list of
> supported image formats.
> This fixes writing the cover image to m4v/m4a and other container
> formats that do not allow these codecs as a track.
> Signed-off-by: Timo Teräs <timo.teras at iki.fi>
> This replaces the previous patch:
> PATCH] [RFC] avformat/movenc: support covert images for ipod muxer
> It appears that query_codec() is used only to figure out which stream
> to select as the default copy stream. Fixing it properly for movenc
> attached pictures is difficult, as the stream disposition would need
> to be passed through the call stack. Additionally, it would make sense
> to rewrite the selection logic open_output_file() to not use APIC
> special tag and be based on the input/output dispositions.
> So this just adds proper handling of the codecs in mov_init() phase
> which fixes most of the usability issues.
> Please backport to 4.0 stable branch too.
> Changes in v2:
> - map the codec to the actual covr image tag value and
> use that when writing the cover image instead of doing
> a new switch over the codec
> libavformat/movenc.c | 30 ++++++++++++------------------
> 1 file changed, 12 insertions(+), 18 deletions(-)
this seems to break:
./ffmpeg -i GF9720Repeal20the20Eighth20with20Helen20Linehan.m4a -t 1 test.mov
[mov @ 0x2f45140] Could not find tag for codec h264 in stream #0, codec not currently supported in container
Could not write header for output file #0 (incorrect codec parameters ?): Invalid argument
Error initializing output stream 0:1 --
input can be found here: https://guiltyfeminist.libsyn.com/97-repeal-the-eighth-with-helen-linehan
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
If you think the mosad wants you dead since a long time then you are either
wrong or dead since a long time.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: not available
More information about the ffmpeg-devel