diff options
author | René Scharfe <rene.scharfe@lsrfire.ath.cx> | 2012-05-09 22:21:56 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2012-05-09 14:03:03 -0700 |
commit | 0af596c6ffba19bff4362ac532be35abd67d2630 (patch) | |
tree | a4fa21c8965f37662b26a7d3f6383b0dfffbb0c4 | |
parent | xdiff: add hunk_func() (diff) | |
download | tgif-0af596c6ffba19bff4362ac532be35abd67d2630.tar.xz |
blame: use hunk_func(), part 1
Use blame_chunk_cb() directly as hunk_func() callback, without detour
through xdi_diff_hunks().
Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r-- | builtin/blame.c | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/builtin/blame.c b/builtin/blame.c index 324d476abf..9ae5cf3a56 100644 --- a/builtin/blame.c +++ b/builtin/blame.c @@ -759,12 +759,14 @@ struct blame_chunk_cb_data { long tlno; }; -static void blame_chunk_cb(void *data, long same, long p_next, long t_next) +static int blame_chunk_cb(long start_a, long count_a, + long start_b, long count_b, void *data) { struct blame_chunk_cb_data *d = data; - blame_chunk(d->sb, d->tlno, d->plno, same, d->target, d->parent); - d->plno = p_next; - d->tlno = t_next; + blame_chunk(d->sb, d->tlno, d->plno, start_b, d->target, d->parent); + d->plno = start_a + count_a; + d->tlno = start_b + count_b; + return 0; } /* @@ -781,6 +783,8 @@ static int pass_blame_to_parent(struct scoreboard *sb, struct blame_chunk_cb_data d; xpparam_t xpp; xdemitconf_t xecfg; + xdemitcb_t ecb; + memset(&d, 0, sizeof(d)); d.sb = sb; d.target = target; d.parent = parent; last_in_target = find_last_in_target(sb, target); @@ -795,7 +799,10 @@ static int pass_blame_to_parent(struct scoreboard *sb, xpp.flags = xdl_opts; memset(&xecfg, 0, sizeof(xecfg)); xecfg.ctxlen = 0; - xdi_diff_hunks(&file_p, &file_o, blame_chunk_cb, &d, &xpp, &xecfg); + xecfg.hunk_func = blame_chunk_cb; + memset(&ecb, 0, sizeof(ecb)); + ecb.priv = &d; + xdi_diff(&file_p, &file_o, &xpp, &xecfg, &ecb); /* The rest (i.e. anything after tlno) are the same as the parent */ blame_chunk(sb, d.tlno, d.plno, last_in_target, target, parent); |