]> git.ipfire.org Git - thirdparty/git.git/commit - refs.c
ls-refs: use repository parameter to iterate refs
authorJeff King <peff@peff.net>
Tue, 13 Dec 2022 11:11:10 +0000 (06:11 -0500)
committerJunio C Hamano <gitster@pobox.com>
Tue, 13 Dec 2022 13:16:22 +0000 (22:16 +0900)
commit91e2ab1587d8ee18e3d2978f2b7bc250faf5df8f
tree209986d33cab0ede9918f91ca07e5cc8169160c0
parentc48035d29b4e524aed3a32f0403676f0d9128863
ls-refs: use repository parameter to iterate refs

The ls_refs() function (for the v2 protocol command of the same name)
takes a repository parameter (like all v2 commands), but ignores it. It
should use it to access the refs.

This isn't a bug in practice, since we only call this function when
serving upload-pack from the main repository. But it's an awkward
gotcha, and it causes -Wunused-parameter to complain.

The main reason we don't use the repository parameter is that the ref
iteration interface we call doesn't have a "refs_" variant that takes a
ref_store. However we can easily add one. In fact, since there is only
one other caller (in ref-filter.c), there is no need to maintain the
non-repository wrapper; that caller can just use the_repository. It's
still a long way from consistently using a repository object, but it's
one small step in the right direction.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
ls-refs.c
ref-filter.c
refs.c
refs.h