[FFmpeg-devel] [PATCH] mp3 muxer: write all metadata

[Baptiste Coudurier]

Hi,

On 9/30/09 12:02 AM, Anton Khirnov wrote:
 > [...]
 >
> diff --git a/libavformat/id3v2.c b/libavformat/id3v2.c
> index feccaf7..6663c6a 100644
> --- a/libavformat/id3v2.c
> +++ b/libavformat/id3v2.c
> @@ -244,3 +244,22 @@ const AVMetadataConv ff_id3v2_metadata_conv[] = {
>       { "TSOT", "titlesort"},
>       { 0 }
>   };
> +
> +const char ff_id3v2_tags[][4] = {
> +   { 'T', 'A', 'L', 'B'}, { 'T', 'B', 'P', 'M'}, { 'T', 'C', 'O', 'M'},
> +   { 'T', 'C', 'O', 'N'}, { 'T', 'C', 'O', 'P'}, { 'T', 'D', 'E', 'N'},
> +   { 'T', 'D', 'L', 'Y'}, { 'T', 'D', 'O', 'R'}, { 'T', 'D', 'R', 'C'},
> +   { 'T', 'D', 'R', 'L'}, { 'T', 'D', 'T', 'G'}, { 'T', 'E', 'N', 'C'},
> +   { 'T', 'E', 'X', 'T'}, { 'T', 'F', 'L', 'T'}, { 'T', 'I', 'P', 'L'},
> +   { 'T', 'I', 'T', '1'}, { 'T', 'I', 'T', '2'}, { 'T', 'I', 'T', '3'},
> +   { 'T', 'K', 'E', 'Y'}, { 'T', 'L', 'A', 'N'}, { 'T', 'L', 'E', 'N'},
> +   { 'T', 'M', 'C', 'L'}, { 'T', 'M', 'E', 'D'}, { 'T', 'M', 'O', 'O'},
> +   { 'T', 'O', 'A', 'L'}, { 'T', 'O', 'F', 'N'}, { 'T', 'O', 'L', 'Y'},
> +   { 'T', 'O', 'P', 'E'}, { 'T', 'O', 'W', 'N'}, { 'T', 'P', 'E', '1'},
> +   { 'T', 'P', 'E', '2'}, { 'T', 'P', 'E', '3'}, { 'T', 'P', 'E', '4'},
> +   { 'T', 'P', 'O', 'S'}, { 'T', 'P', 'R', 'O'}, { 'T', 'P', 'U', 'B'},
> +   { 'T', 'R', 'C', 'K'}, { 'T', 'R', 'S', 'N'}, { 'T', 'R', 'S', 'O'},
> +   { 'T', 'S', 'O', 'A'}, { 'T', 'S', 'O', 'P'}, { 'T', 'S', 'O', 'T'},
> +   { 'T', 'S', 'R', 'C'}, { 'T', 'S', 'S', 'E'}, { 'T', 'S', 'S', 'T'},
> +   { 0 }

Personally, I find more readable {'T','O','P','E'} without the spaces, 
but that's not my call.

[...]

-- 
Baptiste COUDURIER
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org