[FFmpeg-cvslog] r17372 - trunk/libavformat/ape.c

benoit subversion
Mon Feb 16 15:44:31 CET 2009


Author: benoit
Date: Mon Feb 16 15:44:31 2009
New Revision: 17372

Log:
Add a context to av_log() calls and extend ape_dumpinfo prototype to do that.

Modified:
   trunk/libavformat/ape.c

Modified: trunk/libavformat/ape.c
==============================================================================
--- trunk/libavformat/ape.c	Mon Feb 16 13:48:45 2009	(r17371)
+++ trunk/libavformat/ape.c	Mon Feb 16 15:44:31 2009	(r17372)
@@ -162,25 +162,25 @@ static void ape_parse_tag(AVFormatContex
 
     val = get_le32(pb);        /* APE tag version */
     if (val > APE_TAG_VERSION) {
-        av_log(NULL, AV_LOG_ERROR, "Unsupported tag version. (>=%d)\n", APE_TAG_VERSION);
+        av_log(s, AV_LOG_ERROR, "Unsupported tag version. (>=%d)\n", APE_TAG_VERSION);
         return;
     }
 
     tag_bytes = get_le32(pb);  /* tag size */
     if (tag_bytes - APE_TAG_FOOTER_BYTES > (1024 * 1024 * 16)) {
-        av_log(NULL, AV_LOG_ERROR, "Tag size is way too big\n");
+        av_log(s, AV_LOG_ERROR, "Tag size is way too big\n");
         return;
     }
 
     fields = get_le32(pb);     /* number of fields */
     if (fields > 65536) {
-        av_log(NULL, AV_LOG_ERROR, "Too many tag fields (%d)\n", fields);
+        av_log(s, AV_LOG_ERROR, "Too many tag fields (%d)\n", fields);
         return;
     }
 
     val = get_le32(pb);        /* flags */
     if (val & APE_TAG_FLAG_IS_HEADER) {
-        av_log(NULL, AV_LOG_ERROR, "APE Tag is a header\n");
+        av_log(s, AV_LOG_ERROR, "APE Tag is a header\n");
         return;
     }
 
@@ -193,15 +193,15 @@ static void ape_parse_tag(AVFormatContex
         ape_tag_read_field(s);
 
 #if ENABLE_DEBUG
-    av_log(NULL, AV_LOG_DEBUG, "\nAPE Tags:\n\n");
-    av_log(NULL, AV_LOG_DEBUG, "title     = %s\n", s->title);
-    av_log(NULL, AV_LOG_DEBUG, "author    = %s\n", s->author);
-    av_log(NULL, AV_LOG_DEBUG, "copyright = %s\n", s->copyright);
-    av_log(NULL, AV_LOG_DEBUG, "comment   = %s\n", s->comment);
-    av_log(NULL, AV_LOG_DEBUG, "album     = %s\n", s->album);
-    av_log(NULL, AV_LOG_DEBUG, "year      = %d\n", s->year);
-    av_log(NULL, AV_LOG_DEBUG, "track     = %d\n", s->track);
-    av_log(NULL, AV_LOG_DEBUG, "genre     = %s\n", s->genre);
+    av_log(s, AV_LOG_DEBUG, "\nAPE Tags:\n\n");
+    av_log(s, AV_LOG_DEBUG, "title     = %s\n", s->title);
+    av_log(s, AV_LOG_DEBUG, "author    = %s\n", s->author);
+    av_log(s, AV_LOG_DEBUG, "copyright = %s\n", s->copyright);
+    av_log(s, AV_LOG_DEBUG, "comment   = %s\n", s->comment);
+    av_log(s, AV_LOG_DEBUG, "album     = %s\n", s->album);
+    av_log(s, AV_LOG_DEBUG, "year      = %d\n", s->year);
+    av_log(s, AV_LOG_DEBUG, "track     = %d\n", s->track);
+    av_log(s, AV_LOG_DEBUG, "genre     = %s\n", s->genre);
 #endif
 }
 
@@ -213,58 +213,58 @@ static int ape_probe(AVProbeData * p)
     return 0;
 }
 
-static void ape_dumpinfo(APEContext * ape_ctx)
+static void ape_dumpinfo(AVFormatContext * s, APEContext * ape_ctx)
 {
 #if ENABLE_DEBUG
     int i;
 
-    av_log(NULL, AV_LOG_DEBUG, "Descriptor Block:\n\n");
-    av_log(NULL, AV_LOG_DEBUG, "magic                = \"%c%c%c%c\"\n", ape_ctx->magic[0], ape_ctx->magic[1], ape_ctx->magic[2], ape_ctx->magic[3]);
-    av_log(NULL, AV_LOG_DEBUG, "fileversion          = %d\n", ape_ctx->fileversion);
-    av_log(NULL, AV_LOG_DEBUG, "descriptorlength     = %d\n", ape_ctx->descriptorlength);
-    av_log(NULL, AV_LOG_DEBUG, "headerlength         = %d\n", ape_ctx->headerlength);
-    av_log(NULL, AV_LOG_DEBUG, "seektablelength      = %d\n", ape_ctx->seektablelength);
-    av_log(NULL, AV_LOG_DEBUG, "wavheaderlength      = %d\n", ape_ctx->wavheaderlength);
-    av_log(NULL, AV_LOG_DEBUG, "audiodatalength      = %d\n", ape_ctx->audiodatalength);
-    av_log(NULL, AV_LOG_DEBUG, "audiodatalength_high = %d\n", ape_ctx->audiodatalength_high);
-    av_log(NULL, AV_LOG_DEBUG, "wavtaillength        = %d\n", ape_ctx->wavtaillength);
-    av_log(NULL, AV_LOG_DEBUG, "md5                  = ");
+    av_log(s, AV_LOG_DEBUG, "Descriptor Block:\n\n");
+    av_log(s, AV_LOG_DEBUG, "magic                = \"%c%c%c%c\"\n", ape_ctx->magic[0], ape_ctx->magic[1], ape_ctx->magic[2], ape_ctx->magic[3]);
+    av_log(s, AV_LOG_DEBUG, "fileversion          = %d\n", ape_ctx->fileversion);
+    av_log(s, AV_LOG_DEBUG, "descriptorlength     = %d\n", ape_ctx->descriptorlength);
+    av_log(s, AV_LOG_DEBUG, "headerlength         = %d\n", ape_ctx->headerlength);
+    av_log(s, AV_LOG_DEBUG, "seektablelength      = %d\n", ape_ctx->seektablelength);
+    av_log(s, AV_LOG_DEBUG, "wavheaderlength      = %d\n", ape_ctx->wavheaderlength);
+    av_log(s, AV_LOG_DEBUG, "audiodatalength      = %d\n", ape_ctx->audiodatalength);
+    av_log(s, AV_LOG_DEBUG, "audiodatalength_high = %d\n", ape_ctx->audiodatalength_high);
+    av_log(s, AV_LOG_DEBUG, "wavtaillength        = %d\n", ape_ctx->wavtaillength);
+    av_log(s, AV_LOG_DEBUG, "md5                  = ");
     for (i = 0; i < 16; i++)
-         av_log(NULL, AV_LOG_DEBUG, "%02x", ape_ctx->md5[i]);
-    av_log(NULL, AV_LOG_DEBUG, "\n");
+         av_log(s, AV_LOG_DEBUG, "%02x", ape_ctx->md5[i]);
+    av_log(s, AV_LOG_DEBUG, "\n");
 
-    av_log(NULL, AV_LOG_DEBUG, "\nHeader Block:\n\n");
+    av_log(s, AV_LOG_DEBUG, "\nHeader Block:\n\n");
 
-    av_log(NULL, AV_LOG_DEBUG, "compressiontype      = %d\n", ape_ctx->compressiontype);
-    av_log(NULL, AV_LOG_DEBUG, "formatflags          = %d\n", ape_ctx->formatflags);
-    av_log(NULL, AV_LOG_DEBUG, "blocksperframe       = %d\n", ape_ctx->blocksperframe);
-    av_log(NULL, AV_LOG_DEBUG, "finalframeblocks     = %d\n", ape_ctx->finalframeblocks);
-    av_log(NULL, AV_LOG_DEBUG, "totalframes          = %d\n", ape_ctx->totalframes);
-    av_log(NULL, AV_LOG_DEBUG, "bps                  = %d\n", ape_ctx->bps);
-    av_log(NULL, AV_LOG_DEBUG, "channels             = %d\n", ape_ctx->channels);
-    av_log(NULL, AV_LOG_DEBUG, "samplerate           = %d\n", ape_ctx->samplerate);
+    av_log(s, AV_LOG_DEBUG, "compressiontype      = %d\n", ape_ctx->compressiontype);
+    av_log(s, AV_LOG_DEBUG, "formatflags          = %d\n", ape_ctx->formatflags);
+    av_log(s, AV_LOG_DEBUG, "blocksperframe       = %d\n", ape_ctx->blocksperframe);
+    av_log(s, AV_LOG_DEBUG, "finalframeblocks     = %d\n", ape_ctx->finalframeblocks);
+    av_log(s, AV_LOG_DEBUG, "totalframes          = %d\n", ape_ctx->totalframes);
+    av_log(s, AV_LOG_DEBUG, "bps                  = %d\n", ape_ctx->bps);
+    av_log(s, AV_LOG_DEBUG, "channels             = %d\n", ape_ctx->channels);
+    av_log(s, AV_LOG_DEBUG, "samplerate           = %d\n", ape_ctx->samplerate);
 
-    av_log(NULL, AV_LOG_DEBUG, "\nSeektable\n\n");
+    av_log(s, AV_LOG_DEBUG, "\nSeektable\n\n");
     if ((ape_ctx->seektablelength / sizeof(uint32_t)) != ape_ctx->totalframes) {
-        av_log(NULL, AV_LOG_DEBUG, "No seektable\n");
+        av_log(s, AV_LOG_DEBUG, "No seektable\n");
     } else {
         for (i = 0; i < ape_ctx->seektablelength / sizeof(uint32_t); i++) {
             if (i < ape_ctx->totalframes - 1) {
-                av_log(NULL, AV_LOG_DEBUG, "%8d   %d (%d bytes)\n", i, ape_ctx->seektable[i], ape_ctx->seektable[i + 1] - ape_ctx->seektable[i]);
+                av_log(s, AV_LOG_DEBUG, "%8d   %d (%d bytes)\n", i, ape_ctx->seektable[i], ape_ctx->seektable[i + 1] - ape_ctx->seektable[i]);
             } else {
-                av_log(NULL, AV_LOG_DEBUG, "%8d   %d\n", i, ape_ctx->seektable[i]);
+                av_log(s, AV_LOG_DEBUG, "%8d   %d\n", i, ape_ctx->seektable[i]);
             }
         }
     }
 
-    av_log(NULL, AV_LOG_DEBUG, "\nFrames\n\n");
+    av_log(s, AV_LOG_DEBUG, "\nFrames\n\n");
     for (i = 0; i < ape_ctx->totalframes; i++)
-        av_log(NULL, AV_LOG_DEBUG, "%8d   %8lld %8d (%d samples)\n", i, ape_ctx->frames[i].pos, ape_ctx->frames[i].size, ape_ctx->frames[i].nblocks);
+        av_log(s, AV_LOG_DEBUG, "%8d   %8lld %8d (%d samples)\n", i, ape_ctx->frames[i].pos, ape_ctx->frames[i].size, ape_ctx->frames[i].nblocks);
 
-    av_log(NULL, AV_LOG_DEBUG, "\nCalculated information:\n\n");
-    av_log(NULL, AV_LOG_DEBUG, "junklength           = %d\n", ape_ctx->junklength);
-    av_log(NULL, AV_LOG_DEBUG, "firstframe           = %d\n", ape_ctx->firstframe);
-    av_log(NULL, AV_LOG_DEBUG, "totalsamples         = %d\n", ape_ctx->totalsamples);
+    av_log(s, AV_LOG_DEBUG, "\nCalculated information:\n\n");
+    av_log(s, AV_LOG_DEBUG, "junklength           = %d\n", ape_ctx->junklength);
+    av_log(s, AV_LOG_DEBUG, "firstframe           = %d\n", ape_ctx->firstframe);
+    av_log(s, AV_LOG_DEBUG, "totalsamples         = %d\n", ape_ctx->totalsamples);
 #endif
 }
 
@@ -403,7 +403,7 @@ static int ape_read_header(AVFormatConte
     }
 
 
-    ape_dumpinfo(ape);
+    ape_dumpinfo(s, ape);
 
     /* try to read APE tags */
     if (!url_is_streamed(pb)) {




More information about the ffmpeg-cvslog mailing list