[FFmpeg-devel] [PATCH] lavf: Make id3v1_create_tag a lavf internal function
James Almer
jamrial at gmail.com
Sat Jul 13 22:12:02 CEST 2013
On 13/07/13 4:29 PM, Paul B Mahol wrote:
> On 7/13/13, James Almer <jamrial at gmail.com> wrote:
>> There are more containers than just mp3 with support for id3v1,
>> and we currently have muxer for at least one of them
>>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>> libavformat/Makefile | 3 ++-
>> libavformat/id3v1.h | 6 +++++
>> libavformat/id3v1enc.c | 73
>> ++++++++++++++++++++++++++++++++++++++++++++++++++
>> libavformat/mp3enc.c | 52 ++++-------------------------------
>> 4 files changed, 86 insertions(+), 48 deletions(-)
>> create mode 100644 libavformat/id3v1enc.c
>>
>
> What other muxer(s) are going to use this?
ADTS is one. I was planning to add an avoption for it after the APE tag one is
committed, or i can send an updated patch adding both at the same time instead.
Then there's apparently tta, musepack and wavpack that also support it.
More information about the ffmpeg-devel
mailing list