diff options
| author | Daniel Barkalow <barkalow@iabervon.org> | 2008-04-26 19:53:09 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2008-04-27 00:36:17 (GMT) |
| commit | c13b2633f49e3e61b37973204793a4d9ef981175 (patch) | |
| tree | 54f173438e506190b6bbc02f574ec0ab6b8339d7 /http-walker.c | |
| parent | 36c79d2bf893b9957688a6c8c13cc0bf0589e596 (diff) | |
| download | git-c13b2633f49e3e61b37973204793a4d9ef981175.zip git-c13b2633f49e3e61b37973204793a4d9ef981175.tar.gz git-c13b2633f49e3e61b37973204793a4d9ef981175.tar.bz2 | |
Make walker.fetch_ref() take a struct ref.
This simplifies a few things, makes a few things slightly more
complicated, but, more importantly, allows that, when struct ref can
represent a symref, http_fetch_ref() can return one.
Incidentally makes the string that http_fetch_ref() gets include "refs/"
(if appropriate), because that's how the name field of struct ref works.
As far as I can tell, the usage in walker:interpret_target() wouldn't have
worked previously, if it ever would have been used, which it wouldn't
(since the fetch process uses the hash instead of the name of the ref
there).
Signed-off-by: Daniel Barkalow <barkalow@iabervon.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'http-walker.c')
| -rw-r--r-- | http-walker.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/http-walker.c b/http-walker.c index 7bda34d..99f397e 100644 --- a/http-walker.c +++ b/http-walker.c @@ -888,10 +888,10 @@ static int fetch(struct walker *walker, unsigned char *sha1) data->alt->base); } -static int fetch_ref(struct walker *walker, char *ref, unsigned char *sha1) +static int fetch_ref(struct walker *walker, struct ref *ref) { struct walker_data *data = walker->data; - return http_fetch_ref(data->alt->base, ref, sha1); + return http_fetch_ref(data->alt->base, ref); } static void cleanup(struct walker *walker) |