FS#11273 - Format FM frequency depending on the regional settings
Attached to Project:
Rockbox
Opened by Alexander Levin (fml2) - Friday, 14 May 2010, 09:00 GMT
Last edited by Alexander Levin (fml2) - Saturday, 15 May 2010, 21:58 GMT
Opened by Alexander Levin (fml2) - Friday, 14 May 2010, 09:00 GMT
Last edited by Alexander Levin (fml2) - Saturday, 15 May 2010, 21:58 GMT
|
DetailsIn 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.
|
This task depends upon
Closed by Alexander Levin (fml2)
Saturday, 15 May 2010, 21:58 GMT
Reason for closing: Accepted
Additional comments about closing: Committed in r26069
Saturday, 15 May 2010, 21:58 GMT
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.