Rockbox mail archiveSubject: 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. KernighanReceived on 2009-11-03