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