[sheepdog] [PATCH v5 05/16] sheep: store.c don't include farm.h
MORITA Kazutaka
morita.kazutaka at gmail.com
Mon May 20 21:08:02 CEST 2013
At Mon, 20 May 2013 15:50:35 +0800,
Kai Zhang wrote:
>
> Signed-off-by: Kai Zhang <kyle at zelin.io>
> ---
> sheep/sheep_priv.h | 6 +++---
> sheep/store.c | 1 -
> 2 files changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/sheep/sheep_priv.h b/sheep/sheep_priv.h
> index b66f348..1c33a3d 100644
> --- a/sheep/sheep_priv.h
> +++ b/sheep/sheep_priv.h
> @@ -26,6 +26,9 @@
> #include "rbtree.h"
> #include "strbuf.h"
>
> +#define HEX_LEN 40
> +#define NAME_LEN HEX_LEN
> +
Please use more descriptive name because this header file is included all the
sheep source files.
How about SHA1_HEX_LEN and SHA1_NAME_LEN?
Thanks,
Kazutaka
More information about the sheepdog
mailing list