[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: KDE/kdepim/kleopatra/crypto
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2008-02-19 4:17:53
Message-ID: 200802182117.53685.aseigo () kde ! org
[Download RAW message or body]


On Monday 18 February 2008, Ingo Klöcker wrote:
> Sure, but I claim that all KDE developers will be able to understand the
> Q_FOREACH version at first sight because similar constructs are used
> everywhere else in KDE.

+1

> OTOH, I further claim that almost all KDE 
> developers will have to consult the STL documentation in order to
> figure out wtf the STL version does. 

probably, yes

> Anyway, it's your code. 

yes

> You will  have to maintain it. 

not forever. eventually we all move on one way or another. leaving code behind 
that is obfuscated for the masses is just disrespectful of those who will 
come after us.

this is also why we do {}s on every conditional.

Marc, you may be brilliant. those who come after/around you may not be 
brilliant in all the same ways.

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech

["signature.asc" (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic