summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2018-04-26 18:05:51 +0200
committerJan Ekström <jeebjp@gmail.com>2018-04-29 02:21:32 +0300
commit137e34e3e9c3ab63e28c62be294d67f5668caf09 (patch)
tree5f020a9853a3d57c681773d320636b3ac6c4e35f
parent0e340ce804c83acd7cfe13f594dfdc234cf56981 (diff)
downloadmpv-137e34e3e9c3ab63e28c62be294d67f5668caf09.tar.bz2
mpv-137e34e3e9c3ab63e28c62be294d67f5668caf09.tar.xz
demux_mkv: adjust log verbosity levels
With -v -v ("debug" level), which is the default for --log-file, this would log every damn Matroska EBML element and some other uninteresting things, which was very noisy. Adjust the log levels to make them less noisy. Also, change some log calls to MP_ERR for things which are actually errors.
-rw-r--r--demux/demux_mkv.c18
-rw-r--r--demux/ebml.c63
2 files changed, 41 insertions, 40 deletions
diff --git a/demux/demux_mkv.c b/demux/demux_mkv.c
index f836a6eff9..bad5af991b 100644
--- a/demux/demux_mkv.c
+++ b/demux/demux_mkv.c
@@ -356,7 +356,7 @@ static bstr demux_mkv_decode(struct mp_log *log, mkv_track_t *track,
dest = NULL;
goto error;
}
- mp_dbg(log, "lzo decompression buffer too small.\n");
+ mp_trace(log, "lzo decompression buffer too small.\n");
if (dstlen >= maxlen / 2) {
talloc_free(dest);
dest = NULL;
@@ -894,11 +894,11 @@ static int demux_mkv_read_cues(demuxer_t *demuxer)
cue_index_add(demuxer, trackpos->cue_track, pos,
time, trackpos->cue_duration);
mkv_d->index_has_durations |= trackpos->n_cue_duration > 0;
- MP_DBG(demuxer, "|+ found cue point for track %"PRIu64": "
- "timecode %"PRIu64", filepos: %"PRIu64""
- "offset %"PRIu64", duration %"PRIu64"\n",
- trackpos->cue_track, time, pos,
- trackpos->cue_relative_position, trackpos->cue_duration);
+ MP_TRACE(demuxer, "|+ found cue point for track %"PRIu64": "
+ "timecode %"PRIu64", filepos: %"PRIu64""
+ "offset %"PRIu64", duration %"PRIu64"\n",
+ trackpos->cue_track, time, pos,
+ trackpos->cue_relative_position, trackpos->cue_duration);
}
}
@@ -1232,8 +1232,8 @@ static int demux_mkv_read_seekhead(demuxer_t *demuxer)
continue;
}
uint64_t pos = seek->seek_position + mkv_d->segment_start;
- MP_DBG(demuxer, "Element 0x%"PRIx32" at %"PRIu64".\n",
- seek->seek_id, pos);
+ MP_TRACE(demuxer, "Element 0x%"PRIx32" at %"PRIu64".\n",
+ seek->seek_id, pos);
get_header_element(demuxer, seek->seek_id, pos);
}
out:
@@ -1938,7 +1938,7 @@ static int read_ebml_header(demuxer_t *demuxer)
"Assuming \"matroska\".\n");
} else if (strcmp(ebml_master.doc_type, "matroska") != 0
&& strcmp(ebml_master.doc_type, "webm") != 0) {
- MP_DBG(demuxer, "no head found\n");
+ MP_TRACE(demuxer, "no head found\n");
talloc_free(parse_ctx.talloc_ctx);
return 0;
}
diff --git a/demux/ebml.c b/demux/ebml.c
index a64c1dd99c..44fa0c0793 100644
--- a/demux/ebml.c
+++ b/demux/ebml.c
@@ -327,7 +327,7 @@ static void ebml_parse_element(struct ebml_parse_ctx *ctx, void *target,
{
assert(type->type == EBML_TYPE_SUBELEMENTS);
assert(level < 8);
- MP_DBG(ctx, "%.*sParsing element %s\n", level, " ", type->name);
+ MP_TRACE(ctx, "%.*sParsing element %s\n", level, " ", type->name);
char *s = target;
uint8_t *end = data + size;
@@ -340,7 +340,7 @@ static void ebml_parse_element(struct ebml_parse_ctx *ctx, void *target,
if (len > end - p)
goto past_end_error;
if (len < 0) {
- MP_DBG(ctx, "Error parsing subelement id\n");
+ MP_ERR(ctx, "Error parsing subelement id\n");
goto other_error;
}
p += len;
@@ -348,7 +348,7 @@ static void ebml_parse_element(struct ebml_parse_ctx *ctx, void *target,
if (len > end - p)
goto past_end_error;
if (len < 0) {
- MP_DBG(ctx, "Error parsing subelement length\n");
+ MP_ERR(ctx, "Error parsing subelement length\n");
goto other_error;
}
p += len;
@@ -368,7 +368,7 @@ static void ebml_parse_element(struct ebml_parse_ctx *ctx, void *target,
if (length > end - p) {
if (field_idx >= 0 && type->fields[field_idx].desc->type
!= EBML_TYPE_SUBELEMENTS) {
- MP_DBG(ctx, "Subelement content goes "
+ MP_ERR(ctx, "Subelement content goes "
"past end of containing element\n");
goto other_error;
}
@@ -381,7 +381,7 @@ static void ebml_parse_element(struct ebml_parse_ctx *ctx, void *target,
continue;
past_end_error:
- MP_DBG(ctx, "Subelement headers go past end of containing element\n");
+ MP_ERR(ctx, "Subelement headers go past end of containing element\n");
other_error:
ctx->has_errors = true;
end = startp;
@@ -437,20 +437,20 @@ static void ebml_parse_element(struct ebml_parse_ctx *ctx, void *target,
int len;
uint32_t id = ebml_parse_id(data, end - data, &len);
if (len < 0 || len > end - data) {
- MP_DBG(ctx, "Error parsing subelement\n");
+ MP_ERR(ctx, "Error parsing subelement\n");
break;
}
data += len;
uint64_t length = ebml_parse_length(data, end - data, &len);
if (len < 0 || len > end - data) {
- MP_DBG(ctx, "Error parsing subelement length\n");
+ MP_ERR(ctx, "Error parsing subelement length\n");
break;
}
data += len;
if (length > end - data) {
// Try to parse what is possible from inside this partial element
length = end - data;
- MP_DBG(ctx, "Next subelement content goes "
+ MP_ERR(ctx, "Next subelement content goes "
"past end of containing element, will be truncated\n");
}
int field_idx = -1;
@@ -460,16 +460,17 @@ static void ebml_parse_element(struct ebml_parse_ctx *ctx, void *target,
break;
}
if (field_idx < 0) {
- if (id == 0xec)
- MP_DBG(ctx, "%.*sIgnoring Void element "
- "size: %"PRIu64"\n", level+1, " ", length);
- else if (id == 0xbf)
- MP_DBG(ctx, "%.*sIgnoring CRC-32 "
- "element size: %"PRIu64"\n", level+1, " ",
- length);
- else
+ if (id == 0xec) {
+ MP_TRACE(ctx, "%.*sIgnoring Void element "
+ "size: %"PRIu64"\n", level+1, " ", length);
+ } else if (id == 0xbf) {
+ MP_TRACE(ctx, "%.*sIgnoring CRC-32 "
+ "element size: %"PRIu64"\n", level+1, " ",
+ length);
+ } else {
MP_DBG(ctx, "Ignoring unrecognized "
"subelement. ID: %x size: %"PRIu64"\n", id, length);
+ }
data += length;
continue;
}
@@ -485,20 +486,20 @@ static void ebml_parse_element(struct ebml_parse_ctx *ctx, void *target,
continue;
}
if (*countptr > 0 && !multiple) {
- MP_DBG(ctx, "Another subelement of type "
- "%x %s (size: %"PRIu64"). Only one allowed. Ignoring.\n",
- id, ed->name, length);
+ MP_WARN(ctx, "Another subelement of type "
+ "%x %s (size: %"PRIu64"). Only one allowed. Ignoring.\n",
+ id, ed->name, length);
ctx->has_errors = true;
data += length;
continue;
}
- MP_DBG(ctx, "%.*sParsing %x %s size: %"PRIu64
- " value: ", level+1, " ", id, ed->name, length);
+ MP_TRACE(ctx, "%.*sParsing %x %s size: %"PRIu64
+ " value: ", level+1, " ", id, ed->name, length);
char *fieldptr = s + fd->offset;
switch (ed->type) {
case EBML_TYPE_SUBELEMENTS:
- MP_DBG(ctx, "subelements\n");
+ MP_TRACE(ctx, "subelements\n");
char *subelptr;
if (multiple) {
char *array_start = (char *) *(generic_struct **) fieldptr;
@@ -517,29 +518,29 @@ static void ebml_parse_element(struct ebml_parse_ctx *ctx, void *target,
subelptr = (fieldtype *) fieldptr
GETPTR(uintptr, uint64_t);
if (length < 1 || length > 8) {
- MP_DBG(ctx, "uint invalid length %"PRIu64"\n", length);
+ MP_ERR(ctx, "uint invalid length %"PRIu64"\n", length);
goto error;
}
*uintptr = ebml_parse_uint(data, length);
- MP_DBG(ctx, "uint %"PRIu64"\n", *uintptr);
+ MP_TRACE(ctx, "uint %"PRIu64"\n", *uintptr);
break;
case EBML_TYPE_SINT:;
int64_t *sintptr;
GETPTR(sintptr, int64_t);
if (length > 8) {
- MP_DBG(ctx, "sint invalid length %"PRIu64"\n", length);
+ MP_ERR(ctx, "sint invalid length %"PRIu64"\n", length);
goto error;
}
*sintptr = ebml_parse_sint(data, length);
- MP_DBG(ctx, "sint %"PRId64"\n", *sintptr);
+ MP_TRACE(ctx, "sint %"PRId64"\n", *sintptr);
break;
case EBML_TYPE_FLOAT:;
double *floatptr;
GETPTR(floatptr, double);
if (length != 0 && length != 4 && length != 8) {
- MP_DBG(ctx, "float invalid length %"PRIu64"\n", length);
+ MP_ERR(ctx, "float invalid length %"PRIu64"\n", length);
goto error;
}
*floatptr = ebml_parse_float(data, length);
@@ -554,7 +555,7 @@ static void ebml_parse_element(struct ebml_parse_ctx *ctx, void *target,
char **strptr;
GETPTR(strptr, char *);
*strptr = talloc_strndup(ctx->talloc_ctx, data, length);
- MP_DBG(ctx, "string \"%s\"\n", *strptr);
+ MP_TRACE(ctx, "string \"%s\"\n", *strptr);
break;
case EBML_TYPE_BINARY:;
@@ -566,7 +567,7 @@ static void ebml_parse_element(struct ebml_parse_ctx *ctx, void *target,
GETPTR(binptr, struct bstr);
binptr->start = data;
binptr->len = length;
- MP_DBG(ctx, "binary %zd bytes\n", binptr->len);
+ MP_TRACE(ctx, "binary %zd bytes\n", binptr->len);
break;
case EBML_TYPE_EBML_ID:;
@@ -574,10 +575,10 @@ static void ebml_parse_element(struct ebml_parse_ctx *ctx, void *target,
GETPTR(idptr, uint32_t);
*idptr = ebml_parse_id(data, end - data, &len);
if (len != length) {
- MP_DBG(ctx, "ebml_id broken value\n");
+ MP_ERR(ctx, "ebml_id broken value\n");
goto error;
}
- MP_DBG(ctx, "ebml_id %x\n", (unsigned)*idptr);
+ MP_TRACE(ctx, "ebml_id %x\n", (unsigned)*idptr);
break;
default:
abort();