diff options
author | Junio C Hamano <gitster@pobox.com> | 2011-09-09 11:54:58 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2011-09-12 16:41:53 -0700 |
commit | 29753cddc8007ac63754855d026fe5eca8897d04 (patch) | |
tree | 6dfdf6cae4ff4c1729cf0b8dd400b99d6d494b1a /builtin/remote.c | |
parent | e6bcd979684d5a050bcb5d642e761e0ea350c6ec (diff) | |
download | git-29753cddc8007ac63754855d026fe5eca8897d04.tar.gz |
rename "match_refs()" to "match_push_refs()"jc/match-refs-clarify
Yes, there is a warning that says the function is only used by push in big
red letters in front of this function, but it didn't say a more important
thing it should have said: what the function is for and what it does.
Rename it and document it to avoid future confusion.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/remote.c')
-rw-r--r-- | builtin/remote.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/builtin/remote.c b/builtin/remote.c index f2a9c26dc3..f16b5443cc 100644 --- a/builtin/remote.c +++ b/builtin/remote.c @@ -389,8 +389,8 @@ static int get_push_ref_states(const struct ref *remote_refs, local_refs = get_local_heads(); push_map = copy_ref_list(remote_refs); - match_refs(local_refs, &push_map, remote->push_refspec_nr, - remote->push_refspec, MATCH_REFS_NONE); + match_push_refs(local_refs, &push_map, remote->push_refspec_nr, + remote->push_refspec, MATCH_REFS_NONE); states->push.strdup_strings = 1; for (ref = push_map; ref; ref = ref->next) { |