diff options
| author | Guido Cella <guido@guidocella.xyz> | 2024-05-04 13:42:54 +0200 |
|---|---|---|
| committer | sfan5 <sfan5@live.de> | 2024-05-05 13:59:41 +0200 |
| commit | 646f31f196b740bc9031fb82dbc607e56910a518 (patch) | |
| tree | fea7567d8dd6d25b056c7d0ca510e6289a930e4b | |
| parent | d347ee0e916de83e74fc4dca1d35300307ce3f6b (diff) | |
| download | mpv-646f31f196b740bc9031fb82dbc607e56910a518.tar.bz2 mpv-646f31f196b740bc9031fb82dbc607e56910a518.tar.xz | |
sd_ass: don't log if subtitles have unknown duration
Users shouldn't have to care if subtitles' duration is unknown because
the only difference is that decode() sets a duration to packets without
one, if any.
The main difference was code intentionally breaking sub-seek, sub-step,
sub-start and sub-end for unknown duration subtitles for some reason,
but I removed it in 0f2370476b and 185fa9ffc6.
LRC subtitles always have unknown duration so if you use them having
this logged on every song is annoying, so only log it in verbose mode.
| -rw-r--r-- | sub/sd_ass.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sub/sd_ass.c b/sub/sd_ass.c index 41c14623a1..baaf5c6905 100644 --- a/sub/sd_ass.c +++ b/sub/sd_ass.c @@ -420,7 +420,7 @@ static void decode(struct sd *sd, struct demux_packet *packet) if (sd->opts->sub_stretch_durations || packet->duration < 0 || sub_duration == UINT32_MAX) { if (!ctx->duration_unknown) { - MP_WARN(sd, "Subtitle with unknown duration.\n"); + MP_VERBOSE(sd, "Subtitle with unknown duration.\n"); ctx->duration_unknown = true; } sub_duration = UNKNOWN_DURATION; |