The iterator returned by this function only includes references whose names start with the whole prefix, not all of those in `find_containing_dir(prefix)` as the old docstring claimed. This docstring was probably copy-pasted from old ref-cache code, which had the old specification. But now, `cache_ref_iterator_begin()` (from which the files reference iterator gets its values) automatically wraps its output using `prefix_ref_iterator_begin()` when necessary, so it has the stricter behavior. Signed-off-by: Michael Haggerty <mhagger@xxxxxxxxxxxx> --- refs/refs-internal.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/refs/refs-internal.h b/refs/refs-internal.h index 12cf4e4718..e5f6bb2047 100644 --- a/refs/refs-internal.h +++ b/refs/refs-internal.h @@ -515,8 +515,8 @@ typedef int rename_ref_fn(struct ref_store *ref_store, const char *logmsg); /* - * Iterate over the references in the specified ref_store that are - * within find_containing_dir(prefix). If prefix is NULL or the empty + * Iterate over the references in the specified ref_store whose names + * start with the specified prefix. If prefix is NULL or the empty * string, iterate over all references in the submodule. */ typedef struct ref_iterator *ref_iterator_begin_fn( -- 2.11.0