Subject: Re: [PATCH 07/16] hex.h: move some hex-related declarations from cache.h

From: Derrick Stolee <[email-masked]>
Date: Thu, 23 Feb 2023 09:11:25 -0500

On 2/23/2023 3:05 AM, Elijah Newren via GitGitGadget wrote:
> From: Elijah Newren <[email-masked]>
> 
> hex.c contains code for hex-related functions, but for some reason these
> functions were declared in the catch-all cache.h.  Move the function
> declarations into a hex.h header instead.
> 
> This also allows us to remove includes of cache.h from a few C files.

> diff --git a/cache.h b/cache.h
> index daf6150bb3c..d0619ce2e63 100644
> --- a/cache.h
> +++ b/cache.h
> @@ -13,6 +13,7 @@
>  #include "string-list.h"
>  #include "pack-revindex.h"
>  #include "hash.h"
> +#include "hex.h"

Adding this to cache.h is definitely safe, but it makes it so .c
files that consume the moved code do not _need_ to include hex.h
like I think you want to imply at the end of this patch.

What changes if you drop this from your change?

Thanks,
-Stolee