diff options
| author |  Jeff King <peff@peff.net> | 2021-12-03 00:11:40 -0500 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2021-12-04 23:27:53 -0800 |
| commit | 25449450c0d07dfd490906c5114f67452aa18de8 (patch) | |
| tree | 7c5681cadffbedede3703df5ccc0d378d275c05c /xdiff | |
| parent | xdiff: drop CMP_ENV macro from xhistogram (diff) | |
| download | tgif-25449450c0d07dfd490906c5114f67452aa18de8.tar.xz | |
xdiff: drop xpparam_t parameter from histogram cmp_recs()
Since 663c5ad035 (diff histogram: intern strings, 2021-11-17), our
cmp_recs() does not call xdl_recmatch(), and thus no longer needs an
xpparam_t struct from which to get the flags. We can drop the unused
parameter from the function, as well as the macro which wraps it.
There's no functional change here; it's just simplifying things (and
making -Wunused-parameter happier).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'xdiff')
| -rw-r--r-- | xdiff/xhistogram.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/xdiff/xhistogram.c b/xdiff/xhistogram.c index 399cc236d7..80794748b0 100644 --- a/xdiff/xhistogram.c +++ b/xdiff/xhistogram.c @@ -88,15 +88,14 @@ struct region { #define REC(env, s, l) \ (env->xdf##s.recs[l - 1]) -static int cmp_recs(xpparam_t const *xpp, - xrecord_t *r1, xrecord_t *r2) +static int cmp_recs(xrecord_t *r1, xrecord_t *r2) { return r1->ha == r2->ha; } #define CMP(i, s1, l1, s2, l2) \ - (cmp_recs(i->xpp, REC(i->env, s1, l1), REC(i->env, s2, l2))) + (cmp_recs(REC(i->env, s1, l1), REC(i->env, s2, l2))) #define TABLE_HASH(index, side, line) \ XDL_HASHLONG((REC(index->env, side, line))->ha, index->table_bits) |