Re: [PATCH 1/2] builtin/blame.c: struct blame_entry does not need a prev link

David Kastrup d...@gnu.org writes: --- Thanks. At some point during its development I must have thought that having it as a dual-linked list may make it easier when we have to split a block into pieces, but it seems that split_overlap() does not need to look at this information. Needs

Re: [PATCH 1/2] builtin/blame.c: struct blame_entry does not need a prev link

Junio C Hamano gits...@pobox.com writes: David Kastrup d...@gnu.org writes: --- Thanks. At some point during its development I must have thought that having it as a dual-linked list may make it easier when we have to split a block into pieces, but it seems that split_overlap() does not

[PATCH 1/2] builtin/blame.c: struct blame_entry does not need a prev link

Signed-off-by: David Kastrup d...@gnu.org --- builtin/blame.c | 13 ++--- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/builtin/blame.c b/builtin/blame.c index e44a6bb..2195595 100644 --- a/builtin/blame.c +++ b/builtin/blame.c @@ -197,7 +197,6 @@ static void

[PATCH 1/2] builtin/blame.c: struct blame_entry does not need a prev link

--- builtin/blame.c | 13 ++--- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/builtin/blame.c b/builtin/blame.c index e44a6bb..2195595 100644 --- a/builtin/blame.c +++ b/builtin/blame.c @@ -197,7 +197,6 @@ static void drop_origin_blob(struct origin *o) * scoreboard