<br><br><div><span class="gmail_quote">2008/4/21, Michael Jansen <<a href="mailto:kde@michael-jansen.biz">kde@michael-jansen.biz</a>>:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br> As a person with rather negative experiences with kdes ... style obsessed<br> contributors ... i want to add one thing.<br> <br> If i would be more sensible i would have stopped contributing to this project<br> because of some peoples obsession with whitespaces.<br>
 <br> Just a history.<br> <br>  - i got blamed for if( ccc ) it should be if (xxx)<br>  - i got blamed for if (xxx) it has to be if( xxx ) - touched kdepim code<br>  - I got educated that changing NULL to 0 is bad</blockquote>
<div><br>:O<br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">  - I was told that using NULL is against the coding style guide 2 days later<br>
  - I got blamed for code i didn't write nor touch<br> <br> Consider that when working on that code. The rules should be simple and easy<br> to follow even for newcomers. If people of your experience have difficulties<br>
 to agree what the rule is we have problems.<br> <br><br> </blockquote></div><br>Dunno, if somebody asks you to do something silly but harmless you could just do it and move on. Nobody will get harmed. It just happens *all the time* in any open source project that project newcomers are told "thanks for the patch but before we include it please change X, Y, and Z."<br>
Only if different people say different things there is a real problem IMHO - in that situation you should tell the people who tell you different things to first check with each other.<br>BTW, there are two "big" coding styles that I know of: kdelibs style and kdepim style. Other modules may be less uniform.<br>
<br>Yeah, the rules should be simple. But you can't argue with history.<br>