From aced667f48c29a160aa4e5c0a8df037092b28189 Mon Sep 17 00:00:00 2001 From: Michael Sevakis Date: Mon, 18 Sep 2017 06:00:05 -0400 Subject: Undo hacks to meant to get around string formatting limitations The new vuprintf makes unnecessary workarounds due to formatting limitations. I checked grep output for whatever appeared to fit but it's possible I missed some instances because they weren't so obvious. Also, this means sound settings can dynamically work with any number of decimals rather than the current assumption of one or two. Add an ipow() function to help and take advantage of dynamic field width and precision. Consolidate string formatting of sound settings. Change-Id: I46caf534859dfd1916cd440cd25e5206b192fcd8 --- apps/gui/option_select.c | 20 +++----------------- 1 file changed, 3 insertions(+), 17 deletions(-) (limited to 'apps/gui/option_select.c') diff --git a/apps/gui/option_select.c b/apps/gui/option_select.c index 0452467994..f99e833a1e 100644 --- a/apps/gui/option_select.c +++ b/apps/gui/option_select.c @@ -106,23 +106,9 @@ const char *option_get_valuestring(const struct settings_list *setting, } else if ((setting->flags & F_T_SOUND) == F_T_SOUND) { - char sign = ' '; - const char *unit = sound_unit(setting->sound_setting->setting); - int val = sound_val2phys(setting->sound_setting->setting, (int)temp_var); - if (sound_numdecimals(setting->sound_setting->setting)) - { - int integer, dec; - if(val < 0) - { - sign = '-'; - val = abs(val); - } - integer = val / 10; - dec = val % 10; - snprintf(buffer, buf_len, "%c%d.%d %s", sign, integer, dec, unit); - } - else - snprintf(buffer, buf_len, "%d %s", val, unit); + format_sound_value(buffer, buf_len, + setting->sound_setting->setting, + temp_var); } else if ((setting->flags & F_CHOICE_SETTING) == F_CHOICE_SETTING) { -- cgit v1.2.3