Merge commit '9279826008b80daad7446950a821f32033ccd33f'
authorMichael Niedermayer <michaelni@gmx.at>
Thu, 3 Jul 2014 22:50:28 +0000 (00:50 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Thu, 3 Jul 2014 22:50:40 +0000 (00:50 +0200)
* commit '9279826008b80daad7446950a821f32033ccd33f':
  id3v2enc: use a case-insensitive comparison for APIC picture type

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavformat/id3v2enc.c

diff --combined libavformat/id3v2enc.c
index 4d555b3d241c9e083ca41c8c00313845ef9d8210,89181c26a00c063822ff43996b0f22824fa908a1..8b804c4d4a168cdf3eb686ee69da25dcc88aedd4
@@@ -1,20 -1,20 +1,20 @@@
  /*
   * ID3v2 header writer
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
@@@ -26,7 -26,6 +26,7 @@@
  #include "libavutil/intreadwrite.h"
  #include "avformat.h"
  #include "avio.h"
 +#include "avio_internal.h"
  #include "id3v2.h"
  
  static void id3v2_put_size(AVIOContext *pb, int size)
@@@ -111,44 -110,6 +111,44 @@@ static int id3v2_check_write_tag(ID3v2E
      return -1;
  }
  
 +static void id3v2_3_metadata_split_date(AVDictionary **pm)
 +{
 +    AVDictionaryEntry *mtag = NULL;
 +    AVDictionary *dst = NULL;
 +    const char *key, *value;
 +    char year[5] = {0}, day_month[5] = {0};
 +    int i;
 +
 +    while ((mtag = av_dict_get(*pm, "", mtag, AV_DICT_IGNORE_SUFFIX))) {
 +        key = mtag->key;
 +        if (!av_strcasecmp(key, "date")) {
 +            /* split date tag using "YYYY-MM-DD" format into year and month/day segments */
 +            value = mtag->value;
 +            i = 0;
 +            while (value[i] >= '0' && value[i] <= '9') i++;
 +            if (value[i] == '\0' || value[i] == '-') {
 +                av_strlcpy(year, value, sizeof(year));
 +                av_dict_set(&dst, "TYER", year, 0);
 +
 +                if (value[i] == '-' &&
 +                    value[i+1] >= '0' && value[i+1] <= '1' &&
 +                    value[i+2] >= '0' && value[i+2] <= '9' &&
 +                    value[i+3] == '-' &&
 +                    value[i+4] >= '0' && value[i+4] <= '3' &&
 +                    value[i+5] >= '0' && value[i+5] <= '9' &&
 +                    (value[i+6] == '\0' || value[i+6] == ' ')) {
 +                    snprintf(day_month, sizeof(day_month), "%.2s%.2s", value + i + 4, value + i + 1);
 +                    av_dict_set(&dst, "TDAT", day_month, 0);
 +                }
 +            } else
 +                av_dict_set(&dst, key, value, 0);
 +        } else
 +            av_dict_set(&dst, key, mtag->value, 0);
 +    }
 +    av_dict_free(pm);
 +    *pm = dst;
 +}
 +
  void ff_id3v2_start(ID3v2EncContext *id3, AVIOContext *pb, int id3v2_version,
                      const char *magic)
  {
      avio_wb32(pb, 0);
  }
  
 -int ff_id3v2_write_metadata(AVFormatContext *s, ID3v2EncContext *id3)
 +static int write_metadata(AVIOContext *pb, AVDictionary **metadata,
 +                          ID3v2EncContext *id3, int enc)
  {
      AVDictionaryEntry *t = NULL;
 -    int enc = id3->version == 3 ? ID3v2_ENCODING_UTF16BOM :
 -                                  ID3v2_ENCODING_UTF8;
 -
 -    ff_metadata_conv(&s->metadata, ff_id3v2_34_metadata_conv, NULL);
 -    if (id3->version == 4)
 -        ff_metadata_conv(&s->metadata, ff_id3v2_4_metadata_conv, NULL);
 +    int ret;
  
 -    while ((t = av_dict_get(s->metadata, "", t, AV_DICT_IGNORE_SUFFIX))) {
 -        int ret;
 +    ff_metadata_conv(metadata, ff_id3v2_34_metadata_conv, NULL);
 +    if (id3->version == 3)
 +        id3v2_3_metadata_split_date(metadata);
 +    else if (id3->version == 4)
 +        ff_metadata_conv(metadata, ff_id3v2_4_metadata_conv, NULL);
  
 -        if ((ret = id3v2_check_write_tag(id3, s->pb, t, ff_id3v2_tags, enc)) > 0) {
 +    while ((t = av_dict_get(*metadata, "", t, AV_DICT_IGNORE_SUFFIX))) {
 +        if ((ret = id3v2_check_write_tag(id3, pb, t, ff_id3v2_tags, enc)) > 0) {
              id3->len += ret;
              continue;
          }
 -        if ((ret = id3v2_check_write_tag(id3, s->pb, t, id3->version == 3 ?
 -                                               ff_id3v2_3_tags : ff_id3v2_4_tags, enc)) > 0) {
 +        if ((ret = id3v2_check_write_tag(id3, pb, t, id3->version == 3 ?
 +                                         ff_id3v2_3_tags : ff_id3v2_4_tags, enc)) > 0) {
              id3->len += ret;
              continue;
          }
  
          /* unknown tag, write as TXXX frame */
 -        if ((ret = id3v2_put_ttag(id3, s->pb, t->key, t->value, MKBETAG('T', 'X', 'X', 'X'), enc)) < 0)
 +        if ((ret = id3v2_put_ttag(id3, pb, t->key, t->value, MKBETAG('T', 'X', 'X', 'X'), enc)) < 0)
              return ret;
          id3->len += ret;
      }
      return 0;
  }
  
 +static int write_chapter(AVFormatContext *s, ID3v2EncContext *id3, int id, int enc)
 +{
 +    const AVRational time_base = {1, 1000};
 +    AVChapter *ch = s->chapters[id];
 +    uint8_t *dyn_buf = NULL;
 +    AVIOContext *dyn_bc = NULL;
 +    char name[123];
 +    int len, start, end, ret;
 +
 +    if ((ret = avio_open_dyn_buf(&dyn_bc)) < 0)
 +        goto fail;
 +
 +    start = av_rescale_q(ch->start, ch->time_base, time_base);
 +    end   = av_rescale_q(ch->end,   ch->time_base, time_base);
 +
 +    snprintf(name, 122, "ch%d", id);
 +    id3->len += avio_put_str(dyn_bc, name);
 +    avio_wb32(dyn_bc, start);
 +    avio_wb32(dyn_bc, end);
 +    avio_wb32(dyn_bc, 0xFFFFFFFFu);
 +    avio_wb32(dyn_bc, 0xFFFFFFFFu);
 +
 +    if ((ret = write_metadata(dyn_bc, &ch->metadata, id3, enc)) < 0)
 +        goto fail;
 +
 +    len = avio_close_dyn_buf(dyn_bc, &dyn_buf);
 +    id3->len += 16 + ID3v2_HEADER_SIZE;
 +
 +    avio_wb32(s->pb, MKBETAG('C', 'H', 'A', 'P'));
 +    avio_wb32(s->pb, len);
 +    avio_wb16(s->pb, 0);
 +    avio_write(s->pb, dyn_buf, len);
 +
 +fail:
 +    if (dyn_bc && !dyn_buf)
 +        avio_close_dyn_buf(dyn_bc, &dyn_buf);
 +    av_freep(&dyn_buf);
 +
 +    return ret;
 +}
 +
 +int ff_id3v2_write_metadata(AVFormatContext *s, ID3v2EncContext *id3)
 +{
 +    int enc = id3->version == 3 ? ID3v2_ENCODING_UTF16BOM :
 +                                  ID3v2_ENCODING_UTF8;
 +    int i, ret;
 +
 +    if ((ret = write_metadata(s->pb, &s->metadata, id3, enc)) < 0)
 +        return ret;
 +
 +    for (i = 0; i < s->nb_chapters; i++) {
 +        if ((ret = write_chapter(s, id3, i, enc)) < 0)
 +            return ret;
 +    }
 +
 +    return 0;
 +}
 +
  int ff_id3v2_write_apic(AVFormatContext *s, ID3v2EncContext *id3, AVPacket *pkt)
  {
      AVStream *st = s->streams[pkt->stream_index];
      /* get the picture type */
      e = av_dict_get(st->metadata, "comment", NULL, 0);
      for (i = 0; e && i < FF_ARRAY_ELEMS(ff_id3v2_picture_types); i++) {
-         if (strstr(ff_id3v2_picture_types[i], e->value) == ff_id3v2_picture_types[i]) {
+         if (!av_strcasecmp(e->value, ff_id3v2_picture_types[i])) {
              type = i;
              break;
          }
      if ((e = av_dict_get(st->metadata, "title", NULL, 0)))
          desc = e->value;
  
 +    /* use UTF16 only for non-ASCII strings */
 +    if (enc == ID3v2_ENCODING_UTF16BOM && string_is_ascii(desc))
 +        enc = ID3v2_ENCODING_ISO8859;
 +
      /* start writing */
      if (avio_open_dyn_buf(&dyn_buf) < 0)
          return AVERROR(ENOMEM);
      return 0;
  }
  
 -void ff_id3v2_finish(ID3v2EncContext *id3, AVIOContext *pb)
 +void ff_id3v2_finish(ID3v2EncContext *id3, AVIOContext *pb,
 +                     int padding_bytes)
  {
 -    int64_t cur_pos = avio_tell(pb);
 +    int64_t cur_pos;
 +
 +    if (padding_bytes < 0)
 +        padding_bytes = 10;
 +
 +    /* The ID3v2.3 specification states that 28 bits are used to represent the
 +     * size of the whole tag.  Therefore the current size of the tag needs to be
 +     * subtracted from the upper limit of 2^28-1 to clip the value correctly. */
 +    /* The minimum of 10 is an arbitrary amount of padding at the end of the tag
 +     * to fix cover art display with some software such as iTunes, Traktor,
 +     * Serato, Torq. */
 +    padding_bytes = av_clip(padding_bytes, 10, 268435455 - id3->len);
 +    ffio_fill(pb, 0, padding_bytes);
 +    id3->len += padding_bytes;
 +
 +    cur_pos = avio_tell(pb);
      avio_seek(pb, id3->size_pos, SEEK_SET);
      id3v2_put_size(pb, id3->len);
      avio_seek(pb, cur_pos, SEEK_SET);
@@@ -355,7 -238,7 +355,7 @@@ int ff_id3v2_write_simple(struct AVForm
      ff_id3v2_start(&id3, s->pb, id3v2_version, magic);
      if ((ret = ff_id3v2_write_metadata(s, &id3)) < 0)
          return ret;
 -    ff_id3v2_finish(&id3, s->pb);
 +    ff_id3v2_finish(&id3, s->pb, s->metadata_header_padding);
  
      return 0;
  }