diff options
author | Jens Arnold <amiconn@rockbox.org> | 2006-02-19 13:34:12 +0000 |
---|---|---|
committer | Jens Arnold <amiconn@rockbox.org> | 2006-02-19 13:34:12 +0000 |
commit | 237d3c4c4b9d4ae1ee48ab812b83009761811604 (patch) | |
tree | 0222738324467191968fe1d3ee81d7130c390a6c /firmware | |
parent | 1b45b130dcee36b1f9f7965fd0b062789bb99bc0 (diff) |
Adaptive button repeat: adapts repeat rate depending on the ability of the application to cope. Avoids afterscroll and similar effects. * Yield while scrolling through lists.
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@8738 a1c6a512-1295-4272-9138-f99709370657
Diffstat (limited to 'firmware')
-rw-r--r-- | firmware/drivers/button.c | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/firmware/drivers/button.c b/firmware/drivers/button.c index 1cb9472e0f..c4c22641ca 100644 --- a/firmware/drivers/button.c +++ b/firmware/drivers/button.c @@ -365,6 +365,7 @@ static void button_tick(void) static int repeat_speed = REPEAT_INTERVAL_START; static int repeat_count = 0; static bool repeat = false; + static bool post = false; int diff; int btn; @@ -381,7 +382,6 @@ static void button_tick(void) /* only poll every X ticks */ if ( ++tick >= POLL_FREQUENCY ) { - bool post = false; btn = button_read(); /* Find out if a key has been released */ @@ -406,7 +406,8 @@ static void button_tick(void) { if ( repeat ) { - count--; + if (!post) + count--; if (count == 0) { post = true; /* yes we have repeat */ @@ -458,9 +459,18 @@ static void button_tick(void) if ( post ) { if (repeat) - queue_post(&button_queue, BUTTON_REPEAT | btn, NULL); + { + if (queue_empty(&button_queue)) + { + queue_post(&button_queue, BUTTON_REPEAT | btn, NULL); + post = false; + } + } else + { queue_post(&button_queue, btn, NULL); + post = false; + } #ifdef HAVE_REMOTE_LCD if(btn & BUTTON_REMOTE) remote_backlight_on(); |