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: r23491 - jdgordon - apps/gui/skin_engine/

Re: r23491 - jdgordon - apps/gui/skin_engine/

From: Frank Gevaerts <frank_at_gevaerts.be>
Date: Tue, 3 Nov 2009 15:33:38 +0100

On Tue, Nov 03, 2009 at 03:06:29PM +0100, Thomas Martitz wrote:
>> %Rp - does the target have recording?
>> %Rf - samplerate
>> %Re - encoder
>> %Rb - bitrate
>> %Rm - mono/stereo
>
> There's at least 1, if not more, new tags that can properly be expressed
> with an equivalent %St|<setting>| expression.

> I don't really like wasting code on tags that have an existing
> equivalent already. Having multiple ways of expressing the same thing
> can be confusing for WPS/SBS authors too.

I think I agree, although of course dedicated tags provide more compact
(and therefore possible more readable) wpses, so maybe for really common
settings we should have them.

Frank (this time without a typo in my own name!)

-- 
"Debugging is twice as hard as writing the code in the first place.
Therefore, if you write the code as cleverly as possible, you are,
by definition, not smart enough to debug it." - Brian W. Kernighan
Received on 2009-11-03

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