[FFmpeg-cvslog] lavf: use av_fourcc2str() where appropriate
Clément Bœsch
git at videolan.org
Wed Mar 29 15:55:04 EEST 2017
ffmpeg | branch: master | Clément Bœsch <u at pkh.me> | Mon Mar 27 09:45:35 2017 +0200| [bec96a7286bc415dd737c8c8f430f0176999e720] | committer: Clément Bœsch
lavf: use av_fourcc2str() where appropriate
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=bec96a7286bc415dd737c8c8f430f0176999e720
---
libavformat/avidec.c | 10 +++-------
libavformat/cafdec.c | 6 ++----
libavformat/dxa.c | 6 ++++--
libavformat/iff.c | 4 ++--
libavformat/mlvdec.c | 3 ++-
libavformat/mov.c | 5 ++---
libavformat/rmdec.c | 9 ++-------
libavformat/wc3movie.c | 10 ++++------
libavformat/westwood_vqa.c | 5 ++---
9 files changed, 23 insertions(+), 35 deletions(-)
diff --git a/libavformat/avidec.c b/libavformat/avidec.c
index 31c33de..4e694fe 100644
--- a/libavformat/avidec.c
+++ b/libavformat/avidec.c
@@ -117,13 +117,9 @@ static const AVMetadataConv avi_metadata_conv[] = {
static int avi_load_index(AVFormatContext *s);
static int guess_ni_flag(AVFormatContext *s);
-#define print_tag(str, tag, size) \
- av_log(NULL, AV_LOG_TRACE, "pos:%"PRIX64" %s: tag=%c%c%c%c size=0x%x\n", \
- avio_tell(pb), str, tag & 0xff, \
- (tag >> 8) & 0xff, \
- (tag >> 16) & 0xff, \
- (tag >> 24) & 0xff, \
- size)
+#define print_tag(str, tag, size) \
+ av_log(NULL, AV_LOG_TRACE, "pos:%"PRIX64" %s: tag=%s size=0x%x\n", \
+ avio_tell(pb), str, av_fourcc2str(tag), size) \
static inline int get_duration(AVIStream *ast, int len)
{
diff --git a/libavformat/cafdec.c b/libavformat/cafdec.c
index fc85fd9..19939d3 100644
--- a/libavformat/cafdec.c
+++ b/libavformat/cafdec.c
@@ -298,11 +298,9 @@ static int read_header(AVFormatContext *s)
break;
default:
-#define _(x) ((x) >= ' ' ? (x) : ' ')
av_log(s, AV_LOG_WARNING,
- "skipping CAF chunk: %08"PRIX32" (%c%c%c%c), size %"PRId64"\n",
- tag, _(tag>>24), _((tag>>16)&0xFF), _((tag>>8)&0xFF), _(tag&0xFF), size);
-#undef _
+ "skipping CAF chunk: %08"PRIX32" (%s), size %"PRId64"\n",
+ tag, av_fourcc2str(av_bswap32(tag)), size);
case MKBETAG('f','r','e','e'):
if (size < 0)
return AVERROR_INVALIDDATA;
diff --git a/libavformat/dxa.c b/libavformat/dxa.c
index 162838c..5019390 100644
--- a/libavformat/dxa.c
+++ b/libavformat/dxa.c
@@ -171,11 +171,13 @@ static int dxa_read_packet(AVFormatContext *s, AVPacket *pkt)
}
avio_seek(s->pb, c->vidpos, SEEK_SET);
while(!avio_feof(s->pb) && c->frames){
+ uint32_t tag;
if ((ret = avio_read(s->pb, buf, 4)) != 4) {
av_log(s, AV_LOG_ERROR, "failed reading chunk type\n");
return ret < 0 ? ret : AVERROR_INVALIDDATA;
}
- switch(AV_RL32(buf)){
+ tag = AV_RL32(buf);
+ switch (tag) {
case MKTAG('N', 'U', 'L', 'L'):
if(av_new_packet(pkt, 4 + pal_size) < 0)
return AVERROR(ENOMEM);
@@ -217,7 +219,7 @@ static int dxa_read_packet(AVFormatContext *s, AVPacket *pkt)
c->readvid = 0;
return 0;
default:
- av_log(s, AV_LOG_ERROR, "Unknown tag %c%c%c%c\n", buf[0], buf[1], buf[2], buf[3]);
+ av_log(s, AV_LOG_ERROR, "Unknown tag %s\n", av_fourcc2str(tag));
return AVERROR_INVALIDDATA;
}
}
diff --git a/libavformat/iff.c b/libavformat/iff.c
index 29fb7bf..6a09eb0 100644
--- a/libavformat/iff.c
+++ b/libavformat/iff.c
@@ -296,8 +296,8 @@ static int parse_dsd_prop(AVFormatContext *s, AVStream *st, uint64_t eof)
st->codecpar->codec_tag = tag = avio_rl32(pb);
st->codecpar->codec_id = ff_codec_get_id(dsd_codec_tags, tag);
if (!st->codecpar->codec_id) {
- av_log(s, AV_LOG_ERROR, "'%c%c%c%c' compression is not supported\n",
- tag&0xFF, (tag>>8)&0xFF, (tag>>16)&0xFF, (tag>>24)&0xFF);
+ av_log(s, AV_LOG_ERROR, "'%s' compression is not supported\n",
+ av_fourcc2str(tag));
return AVERROR_PATCHWELCOME;
}
break;
diff --git a/libavformat/mlvdec.c b/libavformat/mlvdec.c
index 90c3779..319cd26 100644
--- a/libavformat/mlvdec.c
+++ b/libavformat/mlvdec.c
@@ -242,7 +242,8 @@ static int scan_file(AVFormatContext *avctx, AVStream *vst, AVStream *ast, int f
} else if (type == MKTAG('N','U','L','L')) {
} else if (type == MKTAG('M','L','V','I')) { /* occurs when MLV and Mnn files are concatenated */
} else {
- av_log(avctx, AV_LOG_INFO, "unsupported tag %c%c%c%c, size %u\n", type&0xFF, (type>>8)&0xFF, (type>>16)&0xFF, (type>>24)&0xFF, size);
+ av_log(avctx, AV_LOG_INFO, "unsupported tag %s, size %u\n",
+ av_fourcc2str(type), size);
}
avio_skip(pb, size);
}
diff --git a/libavformat/mov.c b/libavformat/mov.c
index 4034ca5..14b85d0 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -2284,9 +2284,8 @@ int ff_mov_read_stsd_entries(MOVContext *c, AVIOContext *pb, int entries)
id = mov_codec_id(st, format);
av_log(c->fc, AV_LOG_TRACE,
- "size=%"PRId64" 4CC= %c%c%c%c/0x%08x codec_type=%d\n", size,
- (format >> 0) & 0xff, (format >> 8) & 0xff, (format >> 16) & 0xff,
- (format >> 24) & 0xff, format, st->codecpar->codec_type);
+ "size=%"PRId64" 4CC=%s/0x%08x codec_type=%d\n", size,
+ av_fourcc2str(format), format, st->codecpar->codec_type);
if (st->codecpar->codec_type==AVMEDIA_TYPE_VIDEO) {
st->codecpar->codec_id = id;
diff --git a/libavformat/rmdec.c b/libavformat/rmdec.c
index e6a1fe8..88c2b9a 100644
--- a/libavformat/rmdec.c
+++ b/libavformat/rmdec.c
@@ -564,13 +564,8 @@ static int rm_read_header(AVFormatContext *s)
tag = avio_rl32(pb);
tag_size = avio_rb32(pb);
avio_rb16(pb);
- av_log(s, AV_LOG_TRACE, "tag=%c%c%c%c (%08x) size=%d\n",
- (tag ) & 0xff,
- (tag >> 8) & 0xff,
- (tag >> 16) & 0xff,
- (tag >> 24) & 0xff,
- tag,
- tag_size);
+ av_log(s, AV_LOG_TRACE, "tag=%s size=%d\n",
+ av_fourcc2str(tag), tag_size);
if (tag_size < 10 && tag != MKTAG('D', 'A', 'T', 'A'))
goto fail;
switch(tag) {
diff --git a/libavformat/wc3movie.c b/libavformat/wc3movie.c
index d8fc18e..cb4d4d9 100644
--- a/libavformat/wc3movie.c
+++ b/libavformat/wc3movie.c
@@ -150,9 +150,8 @@ static int wc3_read_header(AVFormatContext *s)
break;
default:
- av_log(s, AV_LOG_ERROR, " unrecognized WC3 chunk: %c%c%c%c (0x%02X%02X%02X%02X)\n",
- (uint8_t)fourcc_tag, (uint8_t)(fourcc_tag >> 8), (uint8_t)(fourcc_tag >> 16), (uint8_t)(fourcc_tag >> 24),
- (uint8_t)fourcc_tag, (uint8_t)(fourcc_tag >> 8), (uint8_t)(fourcc_tag >> 16), (uint8_t)(fourcc_tag >> 24));
+ av_log(s, AV_LOG_ERROR, "unrecognized WC3 chunk: %s\n",
+ av_fourcc2str(fourcc_tag));
return AVERROR_INVALIDDATA;
}
@@ -274,9 +273,8 @@ static int wc3_read_packet(AVFormatContext *s,
break;
default:
- av_log (s, AV_LOG_ERROR, " unrecognized WC3 chunk: %c%c%c%c (0x%02X%02X%02X%02X)\n",
- (uint8_t)fourcc_tag, (uint8_t)(fourcc_tag >> 8), (uint8_t)(fourcc_tag >> 16), (uint8_t)(fourcc_tag >> 24),
- (uint8_t)fourcc_tag, (uint8_t)(fourcc_tag >> 8), (uint8_t)(fourcc_tag >> 16), (uint8_t)(fourcc_tag >> 24));
+ av_log(s, AV_LOG_ERROR, "unrecognized WC3 chunk: %s\n",
+ av_fourcc2str(fourcc_tag));
ret = AVERROR_INVALIDDATA;
packet_read = 1;
break;
diff --git a/libavformat/westwood_vqa.c b/libavformat/westwood_vqa.c
index 3635c6a..5a54f13 100644
--- a/libavformat/westwood_vqa.c
+++ b/libavformat/westwood_vqa.c
@@ -144,9 +144,8 @@ static int wsvqa_read_header(AVFormatContext *s)
break;
default:
- av_log (s, AV_LOG_ERROR, " note: unknown chunk seen (%c%c%c%c)\n",
- scratch[0], scratch[1],
- scratch[2], scratch[3]);
+ av_log(s, AV_LOG_ERROR, " note: unknown chunk seen (%s)\n",
+ av_fourcc2str(chunk_tag));
break;
}
More information about the ffmpeg-cvslog
mailing list