On 27 February 2015 at 19:03, Thorsten Zachmann <t.zachm...@zagge.de> wrote:
> Hello,
>
>>
>> I say it should be done now, and be done to 2.9 too
>>
>> And I almost don't think maintainers should be able to say no. The long term
>> goal of having a clean codebase is more important than short term issue
>> with git blame (have a separate permanent checkout before this revision
>> where you can run git blame)
>>
>> After all maintainers and developers come and go. It's important all of our
>> codebase is as easy and consistent to look at as possible - and yeah i know
>> that is an argument for keeping easy history too - sigh
>
> There is also the option -w to git blame that ignores whitespace changes.
> Which should be the majority of changes showing a problem.
>
Hmm, yes and maybe it would be useful as default in .gitconfig.
I am using -w with git diff too (and reviewboard has visible option for it).
One thing, it wouldn't ignore \n, eg.:
- foo() {
+ foo()
+ {
But we cannot have everything.
--
regards, Jaroslaw Staniek
KDE:
: A world-wide network of software engineers, artists, writers, translators
: and facilitators committed to Free Software development - http://kde.org
Calligra Suite:
: A graphic art and office suite - http://calligra.org
Kexi:
: A visual database apps builder - http://calligra.org/kexi
Qt Certified Specialist:
: http://www.linkedin.com/in/jstaniek
_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel
