diff options
author | Mark Wooding <mdw@distorted.org.uk> | 2006-03-28 03:23:31 +0100 |
---|---|---|
committer | Junio C Hamano <junkio@cox.net> | 2006-03-27 18:43:51 -0800 |
commit | acb725772964ee11656543a28c303e9aa6d092c5 (patch) | |
tree | f2b53ddc3d0b5640819a4c6d10ee6c6c6d4ee37e /diff.c | |
parent | 9c48666aa0745c666593bb7383e17f78c43eb36d (diff) | |
download | git-acb725772964ee11656543a28c303e9aa6d092c5.tar.gz git-acb725772964ee11656543a28c303e9aa6d092c5.tar.xz |
xdiff: Show function names in hunk headers.
The speed of the built-in diff generator is nice; but the function names
shown by `diff -p' are /really/ nice. And I hate having to choose. So,
we hack xdiff to find the function names and print them.
xdiff has grown a flag to say whether to dig up the function names. The
builtin_diff function passes this flag unconditionally. I suppose it
could parse GIT_DIFF_OPTS, but it doesn't at the moment. I've also
reintroduced the `function name' into the test suite, from which it was
removed in commit 3ce8f089.
The function names are parsed by a particularly stupid algorithm at the
moment: it just tries to find a line in the `old' file, from before the
start of the hunk, whose first character looks plausible. Still, it's
most definitely a start.
Signed-off-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'diff.c')
-rw-r--r-- | diff.c | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -267,6 +267,7 @@ static void builtin_diff(const char *name_a, ecbdata.label_path = lbl; xpp.flags = XDF_NEED_MINIMAL; xecfg.ctxlen = 3; + xecfg.flags = XDL_EMIT_FUNCNAMES; if (!diffopts) ; else if (!strncmp(diffopts, "--unified=", 10)) |