[PATCH 14/27] sha1_file: allow link_alt_odb_entries to handle arbitrary repositories
From: Stefan BellerActually this also allows read_info_alternates and link_alt_odb_entry to handle arbitrary repositories, but link_alt_odb_entries is the most interesting function in this set of functions, hence the commit subject. These functions span a strongly connected component in the function graph, i.e. the recursive call chain might look like -> link_alt_odb_entries -> link_alt_odb_entry -> read_info_alternates -> link_alt_odb_entries That is why we need to convert all these functions at the same time. Signed-off-by: Jonathan Nieder Signed-off-by: Stefan Beller Signed-off-by: Junio C Hamano Signed-off-by: Nguyễn Thái Ngọc Duy --- object-store.h | 4 sha1_file.c| 36 2 files changed, 20 insertions(+), 20 deletions(-) diff --git a/object-store.h b/object-store.h index 79de470639..3fc461a463 100644 --- a/object-store.h +++ b/object-store.h @@ -18,6 +18,10 @@ struct alternate_object_database { char loose_objects_subdir_seen[256]; struct oid_array loose_objects_cache; + /* +* Path to the alternative object store. If this is a relative path, +* it is relative to the current working directory. +*/ char path[FLEX_ARRAY]; }; #define prepare_alt_odb(r) prepare_alt_odb_##r() diff --git a/sha1_file.c b/sha1_file.c index 0fac75bd31..d38f5cdb0e 100644 --- a/sha1_file.c +++ b/sha1_file.c @@ -390,11 +390,10 @@ static int alt_odb_usable(struct raw_object_store *o, * SHA1, an extra slash for the first level indirection, and the * terminating NUL. */ -#define read_info_alternates(r, rb, d) read_info_alternates_##r(rb, d) -static void read_info_alternates_the_repository(const char *relative_base, - int depth); -#define link_alt_odb_entry(r, e, rb, d, n) link_alt_odb_entry_##r(e, rb, d, n) -static int link_alt_odb_entry_the_repository(const char *entry, +static void read_info_alternates(struct repository *r, +const char *relative_base, +int depth); +static int link_alt_odb_entry(struct repository *r, const char *entry, const char *relative_base, int depth, const char *normalized_objdir) { struct alternate_object_database *ent; @@ -420,7 +419,7 @@ static int link_alt_odb_entry_the_repository(const char *entry, while (pathbuf.len && pathbuf.buf[pathbuf.len - 1] == '/') strbuf_setlen(, pathbuf.len - 1); - if (!alt_odb_usable(the_repository->objects, , normalized_objdir)) { + if (!alt_odb_usable(r->objects, , normalized_objdir)) { strbuf_release(); return -1; } @@ -428,12 +427,12 @@ static int link_alt_odb_entry_the_repository(const char *entry, ent = alloc_alt_odb(pathbuf.buf); /* add the alternate entry */ - *the_repository->objects->alt_odb_tail = ent; - the_repository->objects->alt_odb_tail = &(ent->next); + *r->objects->alt_odb_tail = ent; + r->objects->alt_odb_tail = &(ent->next); ent->next = NULL; /* recursively add alternates */ - read_info_alternates(the_repository, pathbuf.buf, depth + 1); + read_info_alternates(r, pathbuf.buf, depth + 1); strbuf_release(); return 0; @@ -468,12 +467,8 @@ static const char *parse_alt_odb_entry(const char *string, return end; } -#define link_alt_odb_entries(r, a, s, rb, d) \ - link_alt_odb_entries_##r(a, s, rb, d) -static void link_alt_odb_entries_the_repository(const char *alt, - int sep, - const char *relative_base, - int depth) +static void link_alt_odb_entries(struct repository *r, const char *alt, +int sep, const char *relative_base, int depth) { struct strbuf objdirbuf = STRBUF_INIT; struct strbuf entry = STRBUF_INIT; @@ -487,7 +482,7 @@ static void link_alt_odb_entries_the_repository(const char *alt, return; } - strbuf_add_absolute_path(, get_object_directory()); + strbuf_add_absolute_path(, r->objects->objectdir); if (strbuf_normalize_path() < 0) die("unable to normalize object directory: %s", objdirbuf.buf); @@ -496,15 +491,16 @@ static void link_alt_odb_entries_the_repository(const char *alt, alt = parse_alt_odb_entry(alt, sep, ); if (!entry.len) continue; - link_alt_odb_entry(the_repository, entry.buf, + link_alt_odb_entry(r, entry.buf, relative_base, depth, objdirbuf.buf); } strbuf_release(); strbuf_release(); }
Re: [PATCH 14/27] sha1_file: allow link_alt_odb_entries to handle arbitrary repositories
On 02/20, Stefan Beller wrote: > Actually this also allows read_info_alternates and link_alt_odb_entry to > handle arbitrary repositories, but link_alt_odb_entries is the most > interesting function in this set of functions, hence the commit subject. > > These functions span a strongly connected component in the function > graph, i.e. the recursive call chain might look like > > -> link_alt_odb_entries > -> link_alt_odb_entry > -> read_info_alternates > -> link_alt_odb_entries > > That is why we need to convert all these functions at the same time. > > Signed-off-by: Jonathan Nieder> Signed-off-by: Stefan Beller > --- Looks good. > object-store.h | 4 > sha1_file.c| 36 > 2 files changed, 20 insertions(+), 20 deletions(-) > > diff --git a/object-store.h b/object-store.h > index 514ad94287..f283fbdba9 100644 > --- a/object-store.h > +++ b/object-store.h > @@ -18,6 +18,10 @@ struct alternate_object_database { > char loose_objects_subdir_seen[256]; > struct oid_array loose_objects_cache; > > + /* > + * Path to the alternative object store. If this is a relative path, > + * it is relative to the current working directory. > + */ Thanks for adding the comment. > char path[FLEX_ARRAY]; > }; > #define prepare_alt_odb(r) prepare_alt_odb_##r() > diff --git a/sha1_file.c b/sha1_file.c > index c26921df4a..6e5105a252 100644 > --- a/sha1_file.c > +++ b/sha1_file.c > @@ -390,11 +390,10 @@ static int alt_odb_usable(struct raw_object_store *o, > * SHA1, an extra slash for the first level indirection, and the > * terminating NUL. > */ > -#define read_info_alternates(r, rb, d) read_info_alternates_##r(rb, d) > -static void read_info_alternates_the_repository(const char *relative_base, > - int depth); > -#define link_alt_odb_entry(r, e, rb, d, n) link_alt_odb_entry_##r(e, rb, d, > n) > -static int link_alt_odb_entry_the_repository(const char *entry, > +static void read_info_alternates(struct repository *r, > + const char *relative_base, > + int depth); > +static int link_alt_odb_entry(struct repository *r, const char *entry, > const char *relative_base, int depth, const char *normalized_objdir) > { > struct alternate_object_database *ent; > @@ -420,7 +419,7 @@ static int link_alt_odb_entry_the_repository(const char > *entry, > while (pathbuf.len && pathbuf.buf[pathbuf.len - 1] == '/') > strbuf_setlen(, pathbuf.len - 1); > > - if (!alt_odb_usable(_repository->objects, , > normalized_objdir)) { > + if (!alt_odb_usable(>objects, , normalized_objdir)) { > strbuf_release(); > return -1; > } > @@ -428,12 +427,12 @@ static int link_alt_odb_entry_the_repository(const char > *entry, > ent = alloc_alt_odb(pathbuf.buf); > > /* add the alternate entry */ > - *the_repository->objects.alt_odb_tail = ent; > - the_repository->objects.alt_odb_tail = &(ent->next); > + *r->objects.alt_odb_tail = ent; > + r->objects.alt_odb_tail = &(ent->next); > ent->next = NULL; > > /* recursively add alternates */ > - read_info_alternates(the_repository, pathbuf.buf, depth + 1); > + read_info_alternates(r, pathbuf.buf, depth + 1); > > strbuf_release(); > return 0; > @@ -468,12 +467,8 @@ static const char *parse_alt_odb_entry(const char > *string, > return end; > } > > -#define link_alt_odb_entries(r, a, s, rb, d) \ > - link_alt_odb_entries_##r(a, s, rb, d) > -static void link_alt_odb_entries_the_repository(const char *alt, > - int sep, > - const char *relative_base, > - int depth) > +static void link_alt_odb_entries(struct repository *r, const char *alt, > + int sep, const char *relative_base, int depth) > { > struct strbuf objdirbuf = STRBUF_INIT; > struct strbuf entry = STRBUF_INIT; > @@ -487,7 +482,7 @@ static void link_alt_odb_entries_the_repository(const > char *alt, > return; > } > > - strbuf_add_absolute_path(, get_object_directory()); > + strbuf_add_absolute_path(, r->objects.objectdir); > if (strbuf_normalize_path() < 0) > die("unable to normalize object directory: %s", > objdirbuf.buf); > @@ -496,15 +491,16 @@ static void link_alt_odb_entries_the_repository(const > char *alt, > alt = parse_alt_odb_entry(alt, sep, ); > if (!entry.len) > continue; > - link_alt_odb_entry(the_repository, entry.buf, > + link_alt_odb_entry(r, entry.buf, > relative_base, depth, objdirbuf.buf); > } >
[PATCH 14/27] sha1_file: allow link_alt_odb_entries to handle arbitrary repositories
Actually this also allows read_info_alternates and link_alt_odb_entry to handle arbitrary repositories, but link_alt_odb_entries is the most interesting function in this set of functions, hence the commit subject. These functions span a strongly connected component in the function graph, i.e. the recursive call chain might look like -> link_alt_odb_entries -> link_alt_odb_entry -> read_info_alternates -> link_alt_odb_entries That is why we need to convert all these functions at the same time. Signed-off-by: Jonathan NiederSigned-off-by: Stefan Beller --- object-store.h | 4 sha1_file.c| 36 2 files changed, 20 insertions(+), 20 deletions(-) diff --git a/object-store.h b/object-store.h index 514ad94287..f283fbdba9 100644 --- a/object-store.h +++ b/object-store.h @@ -18,6 +18,10 @@ struct alternate_object_database { char loose_objects_subdir_seen[256]; struct oid_array loose_objects_cache; + /* +* Path to the alternative object store. If this is a relative path, +* it is relative to the current working directory. +*/ char path[FLEX_ARRAY]; }; #define prepare_alt_odb(r) prepare_alt_odb_##r() diff --git a/sha1_file.c b/sha1_file.c index c26921df4a..6e5105a252 100644 --- a/sha1_file.c +++ b/sha1_file.c @@ -390,11 +390,10 @@ static int alt_odb_usable(struct raw_object_store *o, * SHA1, an extra slash for the first level indirection, and the * terminating NUL. */ -#define read_info_alternates(r, rb, d) read_info_alternates_##r(rb, d) -static void read_info_alternates_the_repository(const char *relative_base, - int depth); -#define link_alt_odb_entry(r, e, rb, d, n) link_alt_odb_entry_##r(e, rb, d, n) -static int link_alt_odb_entry_the_repository(const char *entry, +static void read_info_alternates(struct repository *r, +const char *relative_base, +int depth); +static int link_alt_odb_entry(struct repository *r, const char *entry, const char *relative_base, int depth, const char *normalized_objdir) { struct alternate_object_database *ent; @@ -420,7 +419,7 @@ static int link_alt_odb_entry_the_repository(const char *entry, while (pathbuf.len && pathbuf.buf[pathbuf.len - 1] == '/') strbuf_setlen(, pathbuf.len - 1); - if (!alt_odb_usable(_repository->objects, , normalized_objdir)) { + if (!alt_odb_usable(>objects, , normalized_objdir)) { strbuf_release(); return -1; } @@ -428,12 +427,12 @@ static int link_alt_odb_entry_the_repository(const char *entry, ent = alloc_alt_odb(pathbuf.buf); /* add the alternate entry */ - *the_repository->objects.alt_odb_tail = ent; - the_repository->objects.alt_odb_tail = &(ent->next); + *r->objects.alt_odb_tail = ent; + r->objects.alt_odb_tail = &(ent->next); ent->next = NULL; /* recursively add alternates */ - read_info_alternates(the_repository, pathbuf.buf, depth + 1); + read_info_alternates(r, pathbuf.buf, depth + 1); strbuf_release(); return 0; @@ -468,12 +467,8 @@ static const char *parse_alt_odb_entry(const char *string, return end; } -#define link_alt_odb_entries(r, a, s, rb, d) \ - link_alt_odb_entries_##r(a, s, rb, d) -static void link_alt_odb_entries_the_repository(const char *alt, - int sep, - const char *relative_base, - int depth) +static void link_alt_odb_entries(struct repository *r, const char *alt, +int sep, const char *relative_base, int depth) { struct strbuf objdirbuf = STRBUF_INIT; struct strbuf entry = STRBUF_INIT; @@ -487,7 +482,7 @@ static void link_alt_odb_entries_the_repository(const char *alt, return; } - strbuf_add_absolute_path(, get_object_directory()); + strbuf_add_absolute_path(, r->objects.objectdir); if (strbuf_normalize_path() < 0) die("unable to normalize object directory: %s", objdirbuf.buf); @@ -496,15 +491,16 @@ static void link_alt_odb_entries_the_repository(const char *alt, alt = parse_alt_odb_entry(alt, sep, ); if (!entry.len) continue; - link_alt_odb_entry(the_repository, entry.buf, + link_alt_odb_entry(r, entry.buf, relative_base, depth, objdirbuf.buf); } strbuf_release(); strbuf_release(); } -static void read_info_alternates_the_repository(const char *relative_base, - int depth) +static void