- Status Closed
- Percent Complete
- Task Type Patches
- Category FM Tuner
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Release 3.4
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
Attached to Project: Rockbox
Opened by Alexander Levin - 2010-05-14
Last edited by Alexander Levin - 2010-05-15
Opened by Alexander Levin - 2010-05-14
Last edited by Alexander Levin - 2010-05-15
FS#11273 - Format FM frequency depending on the regional settings
In some regions, the frequency is changed in 0.1 MHz steps, so that there's no need to display it with two digits after the decimal point. This patch formats the frequency as needed.
fm_freq.diff
(3.8 KiB)
Closed by Alexander Levin
2010-05-15 21:58
Reason for closing: Accepted
Additional comments about closing:
2010-05-15 21:58
Reason for closing: Accepted
Additional comments about closing:
Committed in r26069
looks fine.. can you get rid of the extra {}'s in the case WPS_TOKEN_TUNER_[MIN|MAX|CUR]FREQ cases? they make the switch a bit hard to read
Yes, I will change the formatting function so that it returns the buf. Then all the switch cases can be written shorter.