Re: [PATCH 02/19] object-file: stop using `the_hash_algo`

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Patrick Steinhardt <ps@xxxxxx> writes:

> There are a couple of users of the `the_hash_algo` macro, which
> implicitly depends on `the_repository`. Adapt these callers to not do so
> anymore, either by deriving it from already-available context or by
> using `the_repository->hash_algo`. The latter variant doesn't yet help
> to remove the global dependency, but such users will be adapted in the
> following commits to not use `the_repository` anymore, either.
>

The 'either' doesn't make sense here.

>
> Signed-off-by: Patrick Steinhardt <ps@xxxxxx>
> ---
>  object-file.c | 40 ++++++++++++++++++++++++----------------
>  object-file.h |  1 +
>  2 files changed, 25 insertions(+), 16 deletions(-)
>
> diff --git a/object-file.c b/object-file.c
> index 987cf289420..bc395febc9d 100644
> --- a/object-file.c
> +++ b/object-file.c
> @@ -25,6 +25,7 @@
>  #include "pack.h"
>  #include "packfile.h"
>  #include "path.h"
> +#include "read-cache-ll.h"

I wonder why we add this header.

The rest of the patch looks good.

[snip]

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux