Rockbox

  • Status Closed
  • Percent Complete
    100%
  • 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 fml2 - 2010-05-14
Last edited by fml2 - 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  fml2
2010-05-15 21:58
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Committed in r26069

jdgordon commented on 2010-05-15 09:39

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

fml2 commented on 2010-05-15 10:31

Yes, I will change the formatting function so that it returns the buf. Then all the switch cases can be written shorter.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing