Andreas Rheinhardt:
> If the write_id3v2 option is set, the aiff muxer would write id3v2 tags
> if there is global metadata or if there are attached pics to write.
> Chapters are ignored in this check that precedes writing id3v2 tags.
> Yet 47ac344970f1d6ef149c4b8a883b68cdb15112c2 added support for writing
> chapters as id3v2 tags, so one should check for the existence of chapters,
> too; otherwise the chapters would only be written in case there is
> global metadata or an attached pic.
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com>
> ---
> libavformat/aiffenc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavformat/aiffenc.c b/libavformat/aiffenc.c
> index 0145596bec..88c45df334 100644
> --- a/libavformat/aiffenc.c
> +++ b/libavformat/aiffenc.c
> @@ -49,7 +49,7 @@ static int put_id3v2_tags(AVFormatContext *s,
> AIFFOutputContext *aiff)
> AVIOContext *pb = s->pb;
> AVPacketList *pict_list = aiff->pict_list;
>
> - if (!s->metadata && !aiff->pict_list)
> + if (!s->metadata && !s->nb_chapters && !aiff->pict_list)
> return 0;
>
> avio_wl32(pb, MKTAG('I', 'D', '3', ' '));
>
Will apply the rest of this patchset tomorrow unless there are objections.
- Andreas
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
