dev builds
themes manual
device status forums
mailing lists
IRC bugs
dev guide

Rockbox mail archive

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

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

From: Jonathan Gordon <>
Date: Tue, 3 Nov 2009 11:54:43 -0800

2009/11/3 Thomas Martitz <>:
> On 03.11.2009 19:15, Jonathan Gordon wrote:
>> What is the actual harm in having both? it can obviously be argued
>> either way makes the skin more readable, and there is absolutly no
>> negative impact on adding the extra token (when we run out of 2 letter
>> combos we will be up the creek anyway), and that *entire* commit added
>> under 1000bytes on the worst target (and I highly doubt it was because
>> of the one setting which gets a duplicate tag)
> This is what I meant with that you change your coding style every other day.
> (Some of) The new tags are completely redundant. A few days ago you
> discussed hard to kill the "redundant" (it really isn't) %Vi tag.

There is a big difference here... Removing redundancy in code is
good.. removing user options is bad. And I ended up agreeing that %Vi
was needed.. (and will be more needed once I get more into the
recording skinning stuff!)
Received on 2009-11-03

Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy