[PATCH 0/6] unused function parameter potpourri

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