On 02/20, Stefan Beller wrote:
> Signed-off-by: Stefan Beller <sbeller@xxxxxxxxxx>
Straight forward change, looks good.
> ---
> builtin/am.c | 2 +-
> builtin/clone.c | 2 +-
> builtin/fetch.c | 2 +-
> builtin/merge.c | 2 +-
> builtin/receive-pack.c | 2 +-
> object.c | 6 ++----
> packfile.c | 4 ++--
> packfile.h | 2 +-
> 8 files changed, 10 insertions(+), 12 deletions(-)
>
> diff --git a/builtin/am.c b/builtin/am.c
> index 5bdd2d7578..4762a702e3 100644
> --- a/builtin/am.c
> +++ b/builtin/am.c
> @@ -1859,7 +1859,7 @@ static void am_run(struct am_state *state, int resume)
> */
> if (!state->rebasing) {
> am_destroy(state);
> - close_all_packs();
> + close_all_packs(&the_repository->objects);
> run_command_v_opt(argv_gc_auto, RUN_GIT_CMD);
> }
> }
> diff --git a/builtin/clone.c b/builtin/clone.c
> index 101c27a593..13cfaa6488 100644
> --- a/builtin/clone.c
> +++ b/builtin/clone.c
> @@ -1217,7 +1217,7 @@ int cmd_clone(int argc, const char **argv, const char *prefix)
> transport_disconnect(transport);
>
> if (option_dissociate) {
> - close_all_packs();
> + close_all_packs(&the_repository->objects);
> dissociate_from_references();
> }
>
> diff --git a/builtin/fetch.c b/builtin/fetch.c
> index 8ee998ea2e..4d72efca78 100644
> --- a/builtin/fetch.c
> +++ b/builtin/fetch.c
> @@ -1478,7 +1478,7 @@ int cmd_fetch(int argc, const char **argv, const char *prefix)
>
> string_list_clear(&list, 0);
>
> - close_all_packs();
> + close_all_packs(&the_repository->objects);
>
> argv_array_pushl(&argv_gc_auto, "gc", "--auto", NULL);
> if (verbosity < 0)
> diff --git a/builtin/merge.c b/builtin/merge.c
> index 30264cfd7c..907ae44ab5 100644
> --- a/builtin/merge.c
> +++ b/builtin/merge.c
> @@ -411,7 +411,7 @@ static void finish(struct commit *head_commit,
> * We ignore errors in 'gc --auto', since the
> * user should see them.
> */
> - close_all_packs();
> + close_all_packs(&the_repository->objects);
> run_command_v_opt(argv_gc_auto, RUN_GIT_CMD);
> }
> }
> diff --git a/builtin/receive-pack.c b/builtin/receive-pack.c
> index b2eac79a6e..954fc72c7c 100644
> --- a/builtin/receive-pack.c
> +++ b/builtin/receive-pack.c
> @@ -2027,7 +2027,7 @@ int cmd_receive_pack(int argc, const char **argv, const char *prefix)
> proc.git_cmd = 1;
> proc.argv = argv_gc_auto;
>
> - close_all_packs();
> + close_all_packs(&the_repository->objects);
> if (!start_command(&proc)) {
> if (use_sideband)
> copy_to_sideband(proc.err, -1, NULL);
> diff --git a/object.c b/object.c
> index c76b62572a..34daaf37b3 100644
> --- a/object.c
> +++ b/object.c
> @@ -4,6 +4,7 @@
> #include "tree.h"
> #include "commit.h"
> #include "tag.h"
> +#include "packfile.h"
>
> static struct object **obj_hash;
> static int nr_objs, obj_hash_size;
> @@ -469,8 +470,5 @@ void raw_object_store_clear(struct raw_object_store *o)
>
> while (!list_empty(&o->packed_git_mru))
> list_del(&o->packed_git_mru);
> - /*
> - * TODO: call close_all_packs once migrated to
> - * take an object store argument
> - */
> + close_all_packs(o);
> }
> diff --git a/packfile.c b/packfile.c
> index d41e4c83d0..511a2b0cdf 100644
> --- a/packfile.c
> +++ b/packfile.c
> @@ -311,11 +311,11 @@ static void close_pack(struct packed_git *p)
> close_pack_index(p);
> }
>
> -void close_all_packs(void)
> +void close_all_packs(struct raw_object_store *o)
> {
> struct packed_git *p;
>
> - for (p = the_repository->objects.packed_git; p; p = p->next)
> + for (p = o->packed_git; p; p = p->next)
> if (p->do_not_close)
> die("BUG: want to close pack marked 'do-not-close'");
> else
> diff --git a/packfile.h b/packfile.h
> index a7fca598d6..6a2c57045c 100644
> --- a/packfile.h
> +++ b/packfile.h
> @@ -63,7 +63,7 @@ extern void close_pack_index(struct packed_git *);
>
> extern unsigned char *use_pack(struct packed_git *, struct pack_window **, off_t, unsigned long *);
> extern void close_pack_windows(struct packed_git *);
> -extern void close_all_packs(void);
> +extern void close_all_packs(struct raw_object_store *o);
> extern void unuse_pack(struct pack_window **);
> extern void clear_delta_base_cache(void);
> extern struct packed_git *add_packed_git(const char *path, size_t path_len, int local);
> --
> 2.16.1.291.g4437f3f132-goog
>
--
Brandon Williams
