From 137e34e3e9c3ab63e28c62be294d67f5668caf09 Mon Sep 17 00:00:00 2001 From: wm4 Date: Thu, 26 Apr 2018 18:05:51 +0200 Subject: 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. --- demux/demux_mkv.c | 18 ++++++++-------- demux/ebml.c | 63 ++++++++++++++++++++++++++++--------------------------- 2 files changed, 41 insertions(+), 40 deletions(-) (limited to 'demux') 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(); -- cgit v1.2.3