<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
>> 2) I am personally against of automated whitespace reformatting, because<br>
>> it pollutes history of files without any use. Includes,<br>
>> slots, forward declarations reformattings are ok. Whitespace no.<br>
><br>
><br>
> There is a lot of use in whitespace cleanup: it makes the code<br>
> consistent and readable.<br>
<br>
</span>Yes, we discussed that a bit a while ago. Initially I was critical<br>
too. But whitespace changes can be ignored when generating diff _and_<br>
blame (the -w option). Anyone can add this flag in git options for<br>
diff/blame. I'd do so.<br>
<br>
I'd like to note that we don't have a proven formatting tool for now<br>
for all C++ elements. astyle got broken, and clang formatter can be<br>
better but we need an expert. Missing things may be minor but we can't<br>
make the result worse in problematic areas than the original. Not even<br>
KF5 has a formatting tool, last I checked.<br></blockquote><div><br></div><div>Yes, the last time we discussed it, I raised quite a few issues with the formatter and got a response that "they are limitations of the formatter and cannot be overcome easily". So why we should spend time on it at all if it doesn't solve any real issue? At least in Krita I don't see any real problem with formatting. Yes, there are a few places where it is non-standard, but we fix it when refactoring these lines.<br></div><div class="h5"> <br></div></div><br>-- <br><div class="gmail_signature">Dmitry Kazakov</div>
</div></div>