Re: [PATCH v5 09/15] packfile: split up responsibilities of `reprepare_packed_git()`

> In `reprepare_packed_git()` we perform a couple of operations:
> 
>   - We reload alternate object directories.
> 
>   - We clear the loose object cache.
> 
>   - We reprepare packfiles.
> 
> While the logic is hosted in "packfile.c", it clearly reaches into other
> subsystems that aren't related to packfiles.
> 
> Split up the responsibility and introduce `odb_reprepare()` which now
> becomes responsible for repreparing the whole object database. The
> existing `reprepare_packed_git()` function is refactored accordingly and
> only cares about reloading the packfile store now.
> 
> Signed-off-by: Patrick Steinhardt <ps@pks.im>
> ---

> diff --git a/odb.h b/odb.h
> index 1c998a2478..ef34132c58 100644
> --- a/odb.h
> +++ b/odb.h
> @@ -163,6 +163,12 @@ struct object_database {
>  struct object_database *odb_new(struct repository *repo);
>  void odb_clear(struct object_database *o);
>  
> +/*
> + * Clear caches, reload alternates and then reload object sources so that new
> + * objects may become accessible.
> + */
> +void odb_reprepare(struct object_database *o);

> diff --git a/packfile.h b/packfile.h
> index bf66211986..a85ff607fe 100644
> --- a/packfile.h
> +++ b/packfile.h
> @@ -112,6 +112,14 @@ void packfile_store_free(struct packfile_store *store);
>   */
>  void packfile_store_close(struct packfile_store *store);
>  
> +/*
> + * Clear the packfile caches and try to look up any new packfiles that have
> + * appeared since last preparing the packfiles store.
> + *
> + * This function must be called under the `odb_read_lock()`.
> + */
> +void packfile_store_reprepare(struct packfile_store *store);