[FFmpeg-devel] [PATCH 2/2] Revert "Simplify code by using av_metadata_set2()."

Anton Khirnov anton
Mon Nov 29 17:39:19 CET 2010


On Mon, Nov 29, 2010 at 01:53:37PM +0100, Aurelien Jacobs wrote:
> On Mon, Nov 29, 2010 at 09:41:52AM +0100, Anton Khirnov wrote:
> > This reverts commit r20840.
> > With it, track number is exported as a float, which is not
> > desirable.
> > ---
> >  libavformat/id3v1.c |    8 +++++---
> >  1 files changed, 5 insertions(+), 3 deletions(-)
> > 
> > diff --git a/libavformat/id3v1.c b/libavformat/id3v1.c
> > index c72fca4..dc06b3d 100644
> > --- a/libavformat/id3v1.c
> > +++ b/libavformat/id3v1.c
> > @@ -21,7 +21,6 @@
> >  
> >  #include "id3v1.h"
> >  #include "libavcodec/avcodec.h"
> > -#include "libavutil/avstring.h"
> >  
> >  const char * const ff_id3v1_genre_str[ID3v1_GENRE_MAX + 1] = {
> >        [0] = "Blues",
> > @@ -202,6 +201,7 @@ static void get_string(AVFormatContext *s, const char *key,
> >   */
> >  static int parse_tag(AVFormatContext *s, const uint8_t *buf)
> >  {
> > +    char str[5];
> >      int genre;
> >  
> >      if (!(buf[0] == 'T' &&
> > @@ -213,8 +213,10 @@ static int parse_tag(AVFormatContext *s, const uint8_t *buf)
> >      get_string(s, "album",   buf + 63, 30);
> >      get_string(s, "date",    buf + 93,  4);
> >      get_string(s, "comment", buf + 97, 30);
> > -    if (buf[125] == 0 && buf[126] != 0)
> > -        av_metadata_set2(&s->metadata, "track", av_d2str(buf[126]), AV_METADATA_DONT_STRDUP_VAL);
> > +    if (buf[125] == 0 && buf[126] != 0) {
> > +        snprintf(str, sizeof(str), "%d", buf[126]);
> > +        av_metadata_set(&s->metadata, "track", str);
> > +    }
> 
> This re-introduce usage of av_metadata_set() which is deprecated.
> The right thing to do is to continue using av_metadata_set2() and only
> replace the av_d2str() by snprintf().
Right, applied with this change.

-- 
Anton Khirnov
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101129/4af00606/attachment.pgp>



More information about the ffmpeg-devel mailing list