Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category User Interface
  • 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 freddyb - 2010-02-22
Last edited by teru - 2010-02-25

FS#11047 - Simplify keyboard code

This patch makes no functional changes but simplifies the code by putting each user action into a separate function making the code more maintainable and allowing reuse of code in the future.

   keyboard.c (34 KiB)
   simple_keyboard-v1.diff (21.4 KiB)
Closed by  teru
2010-02-25 07:11
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

Commited partially with some changes.

teru commented on 2010-02-22 08:10

i don’t like this so much. i think it adds too many function. also it increase ram usage by ~280 bytes.

freddyb commented on 2010-02-22 17:04

Are you joking?

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing