Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Drivers
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Version 3.2
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by tomers - 2009-04-10
Last edited by fg - 2009-04-11

FS#10119 - Fix switch case in usb_core.c

This is obviously a typo. I can't see how this code ever compiled…

   fix_switch_case.diff (0.6 KiB)
Closed by  fg
2009-04-11 12:57
Reason for closing:  Fixed
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

Thanks, committed as r20685. I wish I could claim I did this on purpose :)

dreamlayers commented on 2009-04-11 01:34

Yes, that looks like a typo. However, it is valid C, and it doesn't lead to a bug. Case statements are kind of like labels used by goto. You can do unusual things with them, such as http://en.wikipedia.org/wiki/Duff%27s_device

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing