summaryrefslogtreecommitdiffstats
path: root/apps/gui/skin_engine
diff options
context:
space:
mode:
authorMichael Sevakis <jethead71@rockbox.org>2017-09-18 06:00:05 -0400
committerMichael Sevakis <jethead71@rockbox.org>2017-11-21 05:01:14 -0500
commitaced667f48c29a160aa4e5c0a8df037092b28189 (patch)
tree66e48e4a27daaf36f01d7ff1ed6876a7de38b0c0 /apps/gui/skin_engine
parent5c9688961ef9166cec5225db50d5f73691d8292d (diff)
downloadrockbox-aced667f48c29a160aa4e5c0a8df037092b28189.tar.gz
rockbox-aced667f48c29a160aa4e5c0a8df037092b28189.zip
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
Diffstat (limited to 'apps/gui/skin_engine')
-rw-r--r--apps/gui/skin_engine/skin_tokens.c21
1 files changed, 5 insertions, 16 deletions
diff --git a/apps/gui/skin_engine/skin_tokens.c b/apps/gui/skin_engine/skin_tokens.c
index cbf732fe10..1cff83eb9a 100644
--- a/apps/gui/skin_engine/skin_tokens.c
+++ b/apps/gui/skin_engine/skin_tokens.c
@@ -70,6 +70,7 @@
#if CONFIG_TUNER
#include "radio.h"
#include "tuner.h"
+#include "fixedpoint.h"
#endif
#include "list.h"
@@ -432,23 +433,11 @@ const char *get_id3_token(struct wps_token *token, struct mp3entry *id3,
/* Returns buf */
static char *format_freq_MHz(int freq, int freq_step, char *buf, int buf_size)
{
- int scale, div;
- char *fmt;
- if (freq_step < 100000)
- {
- /* Format with two digits after decimal point */
- scale = 10000;
- fmt = "%d.%02d";
- }
- else
- {
- /* Format with one digit after decimal point */
- scale = 100000;
- fmt = "%d.%d";
- }
- div = 1000000 / scale;
+ int decimals = (freq_step < 100000) + 1;
+ int scale = ipow(10, 6 - decimals);
+ int div = 1000000 / scale;
freq = freq / scale;
- snprintf(buf, buf_size, fmt, freq/div, freq%div);
+ snprintf(buf, buf_size, "%d.%.*d", freq/div, decimals, freq%div);
return buf;
}