diff options
| author | Jeff King <peff@peff.net> | 2018-11-02 06:37:18 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2018-11-05 04:14:35 (GMT) |
| commit | 7c61e25fbf1a4a65208be1197940a383f220a1b7 (patch) | |
| tree | 71ec85c998587ae798a4b30f5c154225a5041ec0 /xdiff-interface.c | |
| parent | b135739125e9dc49fafc42d4a8a1956c46329ff1 (diff) | |
| download | git-7c61e25fbf1a4a65208be1197940a383f220a1b7.zip git-7c61e25fbf1a4a65208be1197940a383f220a1b7.tar.gz git-7c61e25fbf1a4a65208be1197940a383f220a1b7.tar.bz2 | |
diff: use hunk callback for word-diff
Our word-diff does not look at the -/+ lines generated by xdiff at all
(because they are not real lines to show the user, but just the
tokenized words split into lines). Instead we use the line numbers from
the hunk headers to index our own data structure.
As a result, our xdi_diff_outf() callback throws away all lines except
hunk headers. We can instead use a hunk callback, which has two
benefits:
1. We don't have to re-parse the generated hunk header line, but can
use the passed parameters directly.
2. By setting our line callback to NULL, we can tell xdiff-interface
that it does not even need to bother generating the other lines,
saving a small amount of work.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'xdiff-interface.c')
| -rw-r--r-- | xdiff-interface.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/xdiff-interface.c b/xdiff-interface.c index b99a578..a23adb3 100644 --- a/xdiff-interface.c +++ b/xdiff-interface.c @@ -95,6 +95,9 @@ static int xdiff_outf(void *priv_, mmbuffer_t *mb, int nbuf) struct xdiff_emit_state *priv = priv_; int i; + if (!priv->line_fn) + return 0; + for (i = 0; i < nbuf; i++) { if (mb[i].ptr[mb[i].size-1] != '\n') { /* Incomplete line */ |