Here are a few small cleanups of unused function parameters. The first five just drop the unused parameters. These are all trivially correct, since otherwise the compiler would complain. But I tried to make sure that dropping was the right thing in each (rather than it being a bug where the parameter should have been used). The final one just uses the parameters for an assertion, following a pattern we've used before. I'll try to cc the individual authors for each patch. [1/6]: xdiff: drop unused mmfile parameters from xdl_do_histogram_diff() [2/6]: log-tree: drop unused commit param in remerge_diff() [3/6]: match_pathname(): drop unused "flags" parameter [4/6]: verify_one_sparse(): drop unused repository parameter [5/6]: reftable: drop unused parameter from reader_seek_linear() [6/6]: reflog: assert PARSE_OPT_NONEG in parse-options callbacks attr.c | 2 +- builtin/reflog.c | 4 ++++ cache-tree.c | 6 ++---- dir.c | 6 ++---- dir.h | 2 +- log-tree.c | 5 ++--- reftable/reader.c | 6 +++--- xdiff/xdiffi.c | 2 +- xdiff/xdiffi.h | 3 +-- xdiff/xhistogram.c | 3 +-- 10 files changed, 18 insertions(+), 21 deletions(-) -Peff