Am 19.08.22 um 10:51 schrieb Jeff King:
> This function has never used its repository parameter since it was
> introduced in 9ad2d5ea71 (sparse-index: loose integration with
> cache_tree_verify(), 2021-03-30). As that commit notes, it may
> eventually be extended further, and that might require looking at a
> repository struct. But it would be easy to add it back later if
> necessary. In the mean time, dropping it makes the code shorter and
> appease -Wunused-parameter.
>
> Signed-off-by: Jeff King <peff@xxxxxxxx>
> ---
> cache-tree.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/cache-tree.c b/cache-tree.c
> index 56db0b5026..c97111cccf 100644
> --- a/cache-tree.c
> +++ b/cache-tree.c
> @@ -857,9 +857,7 @@ int cache_tree_matches_traversal(struct cache_tree *root,
> return 0;
> }
>
> -static void verify_one_sparse(struct repository *r,
> - struct index_state *istate,
> - struct cache_tree *it,
> +static void verify_one_sparse(struct index_state *istate,
This also removes the cache_tree parameter, which has never been used as
well, but is not mentioned in the commit message. A good change, to be
sure.
> struct strbuf *path,
> int pos)
> {
> @@ -910,7 +908,7 @@ static int verify_one(struct repository *r,
> return 1;
>
> if (pos >= 0) {
> - verify_one_sparse(r, istate, it, path, pos);
> + verify_one_sparse(istate, path, pos);
> return 0;
> }
>
