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 | |
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')
-rw-r--r-- | builtin/remote.c | 4 | ||||
-rw-r--r-- | builtin/send-pack.c | 2 |
2 files changed, 3 insertions, 3 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) { diff --git a/builtin/send-pack.c b/builtin/send-pack.c index 87833f4902..e0b8030f2b 100644 --- a/builtin/send-pack.c +++ b/builtin/send-pack.c @@ -509,7 +509,7 @@ int cmd_send_pack(int argc, const char **argv, const char *prefix) flags |= MATCH_REFS_MIRROR; /* match them up */ - if (match_refs(local_refs, &remote_refs, nr_refspecs, refspecs, flags)) + if (match_push_refs(local_refs, &remote_refs, nr_refspecs, refspecs, flags)) return -1; set_ref_status_for_push(remote_refs, args.send_mirror, |