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: Rafaël Carré <rafael.carre_at_gmail.com>
Date: Tue, 30 Mar 2010 01:52:31 +0200

On Mon, 29 Mar 2010 18:34:32 -0500
Paul Louden <paulthenerd_at_gmail.com> 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.

I would tend to agree with you, but then it's difficult to add keymaps
for a target you don't use.

If someone owning such a target will use the feature, then he/she can
participate and add the keymap.

-- 
Rafaël Carré

Received on 2010-03-30

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