[FFmpeg-cvslog] r16952 - in trunk/libavformat: avformat.h avidec.c metadata.c metadata_compat.c

aurel subversion
Mon Feb 2 22:45:56 CET 2009


Author: aurel
Date: Mon Feb  2 22:45:55 2009
New Revision: 16952

Log:
modify the way to pass parameters to av_metadata_set()
This improves code readability and this avoid warnings about discarding
qualifiers from pointer target type.

Modified:
   trunk/libavformat/avformat.h
   trunk/libavformat/avidec.c
   trunk/libavformat/metadata.c
   trunk/libavformat/metadata_compat.c

Modified: trunk/libavformat/avformat.h
==============================================================================
--- trunk/libavformat/avformat.h	Mon Feb  2 22:32:11 2009	(r16951)
+++ trunk/libavformat/avformat.h	Mon Feb  2 22:45:55 2009	(r16952)
@@ -89,10 +89,11 @@ av_metadata_get(AVMetadata *m, const cha
 
 /**
  * sets the given tag in m, overwriting an existing tag.
- * @param tag tag to add to m, key and value will be av_strduped.
+ * @param key tag key to add to m (will be av_strduped).
+ * @param value tag value to add to m (will be av_strduped).
  * @return >= 0 if success otherwise error code that is <0.
  */
-int av_metadata_set(AVMetadata **m, AVMetadataTag tag);
+int av_metadata_set(AVMetadata **pm, const char *key, const char *value);
 
 /**
  * Free all the memory allocated for an AVMetadata struct.

Modified: trunk/libavformat/avidec.c
==============================================================================
--- trunk/libavformat/avidec.c	Mon Feb  2 22:32:11 2009	(r16951)
+++ trunk/libavformat/avidec.c	Mon Feb  2 22:45:55 2009	(r16952)
@@ -229,7 +229,7 @@ static int avi_read_tag(AVFormatContext 
     get_strz(pb, value, sizeof(value));
     url_fseek(pb, i+size, SEEK_SET);
 
-    return av_metadata_set(&s->metadata, (const AVMetadataTag){key, value});
+    return av_metadata_set(&s->metadata, key, value);
 }
 
 static int avi_read_header(AVFormatContext *s, AVFormatParameters *ap)

Modified: trunk/libavformat/metadata.c
==============================================================================
--- trunk/libavformat/metadata.c	Mon Feb  2 22:32:11 2009	(r16951)
+++ trunk/libavformat/metadata.c	Mon Feb  2 22:45:55 2009	(r16952)
@@ -44,10 +44,10 @@ av_metadata_get(AVMetadata *m, const cha
     return NULL;
 }
 
-int av_metadata_set(AVMetadata **pm, AVMetadataTag elem)
+int av_metadata_set(AVMetadata **pm, const char *key, const char *value)
 {
     AVMetadata *m= *pm;
-    AVMetadataTag *tag= av_metadata_get(m, elem.key, NULL, AV_METADATA_MATCH_CASE);
+    AVMetadataTag *tag= av_metadata_get(m, key, NULL, AV_METADATA_MATCH_CASE);
 
     if(!m)
         m=*pm= av_mallocz(sizeof(*m));
@@ -63,10 +63,10 @@ int av_metadata_set(AVMetadata **pm, AVM
         }else
             return AVERROR(ENOMEM);
     }
-    if(elem.value){
-        elem.key  = av_strdup(elem.key  );
-        elem.value= av_strdup(elem.value);
-        m->elems[m->count++]= elem;
+    if(value){
+        m->elems[m->count].key  = av_strdup(key  );
+        m->elems[m->count].value= av_strdup(value);
+        m->count++;
     }
     if(!m->count)
         av_freep(pm);

Modified: trunk/libavformat/metadata_compat.c
==============================================================================
--- trunk/libavformat/metadata_compat.c	Mon Feb  2 22:32:11 2009	(r16951)
+++ trunk/libavformat/metadata_compat.c	Mon Feb  2 22:45:55 2009	(r16952)
@@ -106,7 +106,7 @@ void ff_metadata_demux_compat(AVFormatCo
 
 #define FILL_METADATA(s, key, value) {                                        \
     if (value && *value && !av_metadata_get(s->metadata, #key, NULL, 0))      \
-        av_metadata_set(&s->metadata, (const AVMetadataTag){#key, value});    \
+        av_metadata_set(&s->metadata, #key, value);                           \
     }
 #define FILL_METADATA_STR(s, key)  FILL_METADATA(s, key, s->key)
 #define FILL_METADATA_INT(s, key) {                                           \




More information about the ffmpeg-cvslog mailing list