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: jdgordon: r30599 - in trunk/apps: . gui/skin_engine

Re: jdgordon: r30599 - in trunk/apps: . gui/skin_engine

From: Amaury Pouly <amaury.pouly_at_gmail.com>
Date: Mon, 26 Sep 2011 08:15:21 +0200

Hi All,

> > Furthermore this user setting is implemented in a totally obscure way (a
> > magic file containing a magic number, which isn't checked if it's even
> > plausible). It's pattern that doesn't exist yet, while there's a clear
> > existing implementation of user settings. It's rather hidden without a
> > chance to be discovered by the user. I don't think I need to mention
> > that the manual doesn't tell anything about this.
>

I don't like the way this "setting" is implemented, we have a framework for
that and see no good reason to not use ot.

Regards
Received on 2011-09-26


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