Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: Hotkey patch

Re: Hotkey patch

From: Jeff Goode <jeffg7_at_gmail.com>
Date: Mon, 29 Mar 2010 19:48:47 -0400

On 3/29/2010 19:34, Paul Louden wrote:
> On 3/29/2010 6:19 PM, Jeff Goode wrote:
>> It is an optional feature, same as many others. If a knowledgeable
>> developer wants to switch it on for his target of choice, I've made
>> it very easy to do so. At the same time, my patch consumes no
>> resources and changes nothing if it isn't switched on. That, I
>> think, is as far as I need go.
>>
> The general expectation is that when possible a new feature should be
> enabled for all targets that can support it.
>
> If we get into the habit of just enabling features for one target each
> and make no effort whatsoever, then you'll start to have a wide spread
> of features across targets.
>
The problem is *I don't know* which targets can support it. It's a key
mapping issue, which by definition is target specific. Some targets
have color screens, some have scrollwheels, etc. Some have free ekys
and some do not. It will be up to individuals who want the feature for
a given target to implement it for that target.
Received on 2010-03-30


Page was last modified "Jan 10 2012" The Rockbox Crew
aaa