Commit c4a37885 authored by Diego Biurrun's avatar Diego Biurrun

id3v2: reduce the scope of some non-globally-used symbols/structures

parent 3b78c180
...@@ -54,7 +54,7 @@ const AVMetadataConv ff_id3v2_4_metadata_conv[] = { ...@@ -54,7 +54,7 @@ const AVMetadataConv ff_id3v2_4_metadata_conv[] = {
{ 0 } { 0 }
}; };
const AVMetadataConv ff_id3v2_2_metadata_conv[] = { static const AVMetadataConv id3v2_2_metadata_conv[] = {
{ "TAL", "album"}, { "TAL", "album"},
{ "TCO", "genre"}, { "TCO", "genre"},
{ "TT2", "title"}, { "TT2", "title"},
...@@ -380,7 +380,14 @@ finish: ...@@ -380,7 +380,14 @@ finish:
av_dict_set(m, "date", date, 0); av_dict_set(m, "date", date, 0);
} }
const ID3v2EMFunc ff_id3v2_extra_meta_funcs[] = { typedef struct ID3v2EMFunc {
const char *tag3;
const char *tag4;
void (*read)(AVFormatContext*, AVIOContext*, int, char*, ID3v2ExtraMeta **);
void (*free)();
} ID3v2EMFunc;
static const ID3v2EMFunc id3v2_extra_meta_funcs[] = {
{ "GEO", "GEOB", read_geobtag, free_geobtag }, { "GEO", "GEOB", read_geobtag, free_geobtag },
{ NULL } { NULL }
}; };
...@@ -393,13 +400,12 @@ const ID3v2EMFunc ff_id3v2_extra_meta_funcs[] = { ...@@ -393,13 +400,12 @@ const ID3v2EMFunc ff_id3v2_extra_meta_funcs[] = {
static const ID3v2EMFunc *get_extra_meta_func(const char *tag, int isv34) static const ID3v2EMFunc *get_extra_meta_func(const char *tag, int isv34)
{ {
int i = 0; int i = 0;
while (ff_id3v2_extra_meta_funcs[i].tag3) { while (id3v2_extra_meta_funcs[i].tag3) {
if (!memcmp(tag, if (!memcmp(tag,
(isv34 ? (isv34 ? id3v2_extra_meta_funcs[i].tag4 :
ff_id3v2_extra_meta_funcs[i].tag4 : id3v2_extra_meta_funcs[i].tag3),
ff_id3v2_extra_meta_funcs[i].tag3),
(isv34 ? 4 : 3))) (isv34 ? 4 : 3)))
return &ff_id3v2_extra_meta_funcs[i]; return &id3v2_extra_meta_funcs[i];
i++; i++;
} }
return NULL; return NULL;
...@@ -560,7 +566,7 @@ void ff_id3v2_read_all(AVFormatContext *s, const char *magic, ID3v2ExtraMeta **e ...@@ -560,7 +566,7 @@ void ff_id3v2_read_all(AVFormatContext *s, const char *magic, ID3v2ExtraMeta **e
} }
} while (found_header); } while (found_header);
ff_metadata_conv(&s->metadata, NULL, ff_id3v2_34_metadata_conv); ff_metadata_conv(&s->metadata, NULL, ff_id3v2_34_metadata_conv);
ff_metadata_conv(&s->metadata, NULL, ff_id3v2_2_metadata_conv); ff_metadata_conv(&s->metadata, NULL, id3v2_2_metadata_conv);
ff_metadata_conv(&s->metadata, NULL, ff_id3v2_4_metadata_conv); ff_metadata_conv(&s->metadata, NULL, ff_id3v2_4_metadata_conv);
merge_date(&s->metadata); merge_date(&s->metadata);
} }
......
...@@ -59,13 +59,6 @@ typedef struct ID3v2ExtraMetaGEOB { ...@@ -59,13 +59,6 @@ typedef struct ID3v2ExtraMetaGEOB {
uint8_t *data; uint8_t *data;
} ID3v2ExtraMetaGEOB; } ID3v2ExtraMetaGEOB;
typedef struct ID3v2EMFunc {
const char *tag3;
const char *tag4;
void (*read)(AVFormatContext*, AVIOContext*, int, char*, ID3v2ExtraMeta **);
void (*free)();
} ID3v2EMFunc;
/** /**
* Detect ID3v2 Header. * Detect ID3v2 Header.
* @param buf must be ID3v2_HEADER_SIZE byte long * @param buf must be ID3v2_HEADER_SIZE byte long
...@@ -99,11 +92,8 @@ void ff_id3v2_read_all(AVFormatContext *s, const char *magic, ID3v2ExtraMeta **e ...@@ -99,11 +92,8 @@ void ff_id3v2_read_all(AVFormatContext *s, const char *magic, ID3v2ExtraMeta **e
*/ */
void ff_id3v2_free_extra_meta(ID3v2ExtraMeta **extra_meta); void ff_id3v2_free_extra_meta(ID3v2ExtraMeta **extra_meta);
extern const ID3v2EMFunc ff_id3v2_extra_meta_funcs[];
extern const AVMetadataConv ff_id3v2_34_metadata_conv[]; extern const AVMetadataConv ff_id3v2_34_metadata_conv[];
extern const AVMetadataConv ff_id3v2_4_metadata_conv[]; extern const AVMetadataConv ff_id3v2_4_metadata_conv[];
extern const AVMetadataConv ff_id3v2_2_metadata_conv[];
/** /**
* A list of text information frames allowed in both ID3 v2.3 and v2.4 * A list of text information frames allowed in both ID3 v2.3 and v2.4
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment