Rockbox mail archive
Subject: Re: r23491 - jdgordon - apps/gui/skin_engine/
Re: r23491 - jdgordon - apps/gui/skin_engine/
2009/11/3 Thomas Martitz <thomas.martitz_at_student.htw-berlin.de>:
> 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 was last modified "Jan 10 2012" The Rockbox Crew
|