RepoMirrors
/
mpv
mirror of https://github.com/mpv-player/mpv
1
0
Fork 0
Code Issues Releases Wiki Activity

player/command: truncate anything < 1e-4 in pretty printer 

To avoid switching to scientific notation. Apparently it is "jarring"
for some users.

This preserves status quo from before 9dddfc4f where pretty printer were
truncated to 3 decimal places.
This commit is contained in:
Kacper Michajłow 2023-10-17 02:53:11 +02:00 committed by sfan5
parent 11bbb49bdf
commit 02d009bc5c
2 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
options/m_option.c
View File

@ -1028,7 +1028,12 @@ static char *print_double_7g(const m_option_t *opt, const void *val)
double f = VAL(val); double f = VAL(val);
if (isnan(f)) if (isnan(f))
return print_double(opt, val); return print_double(opt, val);
return talloc_asprintf(NULL, "%.7g", f); // Truncate anything < 1e-4 to avoid switching to scientific notation
if (fabs(f) < 1e-4) {
return talloc_strdup(NULL, "0");
} else {
return talloc_asprintf(NULL, "%.7g", f);
}
} }
static void add_double(const m_option_t *opt, void *val, double add, bool wrap) static void add_double(const m_option_t *opt, void *val, double add, bool wrap)

4
player/command.c
View File

@ -652,8 +652,8 @@ static int mp_property_avsync(void *ctx, struct m_property *prop,
if (!mpctx->ao_chain || !mpctx->vo_chain) if (!mpctx->ao_chain || !mpctx->vo_chain)
return M_PROPERTY_UNAVAILABLE; return M_PROPERTY_UNAVAILABLE;
if (action == M_PROPERTY_PRINT) { if (action == M_PROPERTY_PRINT) {
// Don't print small values resulting from calculation inaccuracies // Truncate anything < 1e-4 to avoid switching to scientific notation
if (fabs(mpctx->last_av_difference) < 1e-5) { if (fabs(mpctx->last_av_difference) < 1e-4) {
*(char **)arg = talloc_strdup(NULL, "0"); *(char **)arg = talloc_strdup(NULL, "0");
} else { } else {
*(char **)arg = talloc_asprintf(NULL, "%+.2g", mpctx->last_av_difference); *(char **)arg = talloc_asprintf(NULL, "%+.2g", mpctx->last_av_difference);