Adding a section "Touching old code" to the Kdelibs Coding Style (was: Re: [PATCH] Set program icon name by KAboutData)
Michael Jansen
kde at michael-jansen.biz
Mon Apr 21 19:53:58 BST 2008
As a person with rather negative experiences with kdes ... style obsessed
contributors ... i want to add one thing.
If i would be more sensible i would have stopped contributing to this project
because of some peoples obsession with whitespaces.
Just a history.
- i got blamed for if( ccc ) it should be if (xxx)
- i got blamed for if (xxx) it has to be if( xxx ) - touched kdepim code
- I got educated that changing NULL to 0 is bad
- I was told that using NULL is against the coding style guide 2 days later
- I got blamed for code i didn't write nor touch
Consider that when working on that code. The rules should be simple and easy
to follow even for newcomers. If people of your experience have difficulties
to agree what the rule is we have problems.
--
Michael Jansen
Available for contract work ( Development / Configuration Management )
http://www.michael-jansen.biz
More information about the kde-core-devel
mailing list