<div class="gmail_quote">On Wed, Nov 9, 2011 at 5:24 PM, David Nolden <span dir="ltr"><<a href="mailto:david.nolden.kdevelop@art-master.de">david.nolden.kdevelop@art-master.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

2011/11/9 Andreas Pakulat <<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>>:<br>
<div class="im">>> Thank you, but I prefer my pretty 4-space indentation in the code<br>
>> that I wrote.<br>
><br>
> Great bike-shedding topic. I agree with Alex on the 4-spaces thing,<br>
> especially since horizontal space is cheap these days.<br>
<br>
</div>Well, some people like to use the increased resources available these<br>
days for useful things, for example for split-views or tiled windows,<br>
instead of wasting it for whitespace.<br></blockquote><div> </div><div>Problems of code width are not solved by removing the indentation spaces but by enforcing to limit the code to 80 columns, or 100. I don't split views myself, so I'm not sure anything on that regard.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"><br>
> IMHO the codebase actually is quite well formatted in general. Things<br>
<br>
</div>It's not terrible, but it's also not good. I can point you at tons of<br>
files which even have 4-space and 2-space indentation mixed. I agree<br>
that it's not such a huge problem for reading, but it's very annoying<br>
when writing small portions of code in many different places, each<br>
with a different kind of formatting which I must adapt to. Therefore,<br>
we should at least have a way to automatically format newly inserted<br>
code. Uncrustify is perfect for this, because it can largely<br>
"normalize" the code. We simply have to define which formatting<br>
configurations we want to use for what portions of our code, and then<br>
define a rule that every committer must format the code before<br>
committing it.<br></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><br>
It's also annoying to always tell new contributors to remove training<br>
whitespaces, add/remove whitespace here and there, or whatever. In<br>
future we can simply tell them "run the formatting script before<br>
before creating the patch", or simply do it by ourselves.<br></blockquote><div>Meh, I don't think trailing spaces are that much of a problem... It's mostly that reviewboard displays them in red neon lights.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
In general, a human shouldn't be forced to do something which a<br>
machine can do better (eg. format the code in a consistent exactly<br>
defined way depending on the current context). Of course we can define<br>
different formattings for different parts of the codebase, although<br>
one consistent formatting would be better.<br></blockquote><div><br></div><div>If we can't even have everyone on board regarding the indentation, I'm not sure how can we agree on a whole coding style for the whole kdev*.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
Greetings, David<br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
</div></div></blockquote></div><br><div>Aleix</div>