Re: [PATCH] commit-slab-decl.h: fix define gaurds
From: Junio C Hamano <hidden>
Date: 2020-05-17 15:44:32
Abhishek Kumar [off-list ref] writes: That's "guard" ;-)
commit-slab was split into commit-slab-decl and commit-slab-impl header
files by a9f1f1f9f8 ("commit-slab.h: code split", 2018-05-19). The
define guards for commit-slab-decl stll use "COMMIT_SLAB_HDR_H", which
no longer matches the file name. Let's fix this.It would have been helpful to explain that the split did not break anything because the other half of the split started using a new and different proprocessor macro before saying "Let's fix". It would have been "broken" that needs "fixing" if two headers inherited the use of the same _HDR_H preprocessor macro, of course. The patch obviously looks good.
quoted hunk
Signed-off-by: Abhishek Kumar <redacted> --- commit-slab-decl.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)diff --git a/commit-slab-decl.h b/commit-slab-decl.h index adc7b46c83..bfbed1516a 100644 --- a/commit-slab-decl.h +++ b/commit-slab-decl.h@@ -1,5 +1,5 @@ -#ifndef COMMIT_SLAB_HDR_H -#define COMMIT_SLAB_HDR_H +#ifndef COMMIT_SLAB_DECL_H +#define COMMIT_SLAB_DECL_H /* allocate ~512kB at once, allowing for malloc overhead */ #ifndef COMMIT_SLAB_SIZE@@ -40,4 +40,4 @@ elemtype *slabname## _peek(struct slabname *s, const struct commit *c) declare_commit_slab(slabname, elemtype); \ declare_commit_slab_prototypes(slabname, elemtype) -#endif /* COMMIT_SLAB_HDR_H */ +#endif /* COMMIT_SLAB_DECL_H */