[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Progress report on git-blame
From: |
Óscar Fuentes |
Subject: |
Re: Progress report on git-blame |
Date: |
Sat, 25 Jan 2014 20:44:56 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux) |
David Kastrup <address@hidden> writes:
>> It seems that there is not a lot of a difference among blaming 20 lines
>> and blaming 1000.
>
> It very much depends on _which_ lines. Basically, lines forming a block
> sticking together when doing a unified diff are not all that expensive
> since they travel in a single data structure. The worst are lots of
> small changes introduced in different branches.
I suspected some effect like that, so before posting I tried other file
segments and most of them required the same time as the one showed, with
the rest requiring less time (7s vs 11s)
>> AFAIK current `blame' functionality on VC (and on git-blame.el) is
>> based on working on the whole file,
>
> Sure?
Not, really, but almost. :-) It seems quite clear that vc-annotate just
displays the output of the underlying VC tool.
> If so, it maybe should at least be made to support git blame
> --incremental usefully.
Of course --incremental can also apply to a line-range blame. From
trying a couple of times it seems that the most recently changed lines
were annotated first, so for Lars' case of checking if some line on the
range was recently changed, he would have the relevant info sooner (but
if all the lines are old, he would have to wait until some line is
annotated to know the fact; in that case there is no significant time
again over non-incremental annotation, so speeding up `git blame' is
worthy in any case.)
- Re: Progress report on git-blame (was: RFC - cleaning up /etc), (continued)
- Re: Progress report on git-blame (was: RFC - cleaning up /etc), Eli Zaretskii, 2014/01/25
- Re: Progress report on git-blame, David Kastrup, 2014/01/25
- Re: Progress report on git-blame, martin rudalics, 2014/01/25
- Re: Progress report on git-blame, David Kastrup, 2014/01/25
- Re: Progress report on git-blame, David Kastrup, 2014/01/25
- Re: Progress report on git-blame, Lars Ingebrigtsen, 2014/01/25
- Re: Progress report on git-blame, David Kastrup, 2014/01/25
- Re: Progress report on git-blame, Óscar Fuentes, 2014/01/25
- Re: Progress report on git-blame, David Kastrup, 2014/01/25
- Re: Progress report on git-blame, Eli Zaretskii, 2014/01/25
- Re: Progress report on git-blame,
Óscar Fuentes <=
- Re: Progress report on git-blame, David Kastrup, 2014/01/25
- Re: Progress report on git-blame, Stefan Monnier, 2014/01/25
- Re: Progress report on git-blame, Aneesh Kumar K.V, 2014/01/26
- Re: Progress report on git-blame, Stefan Monnier, 2014/01/27
- Re: Progress report on git-blame, Stefan Monnier, 2014/01/25
- Re: Progress report on git-blame, Óscar Fuentes, 2014/01/25
- Re: Progress report on git-blame, Stefan Monnier, 2014/01/25
- Re: Progress report on git-blame, Eli Zaretskii, 2014/01/26
- Re: Progress report on git-blame, Stefan Monnier, 2014/01/26
- Re: Progress report on git-blame, Eli Zaretskii, 2014/01/26