[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