diff options
| author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2021-09-27 14:54:25 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2021-09-27 14:47:59 -0700 |
| commit | 9865b6e6a4ca1e895fd473c827cf1822f3bd8249 (patch) | |
| tree | a042ff50d16f1a328592aa46a11311ab66585077 /revision.c | |
| parent | 9d444d9ee019cb795e6a677fbb70daf6ae64a073 (diff) | |
| download | git-9865b6e6a4ca1e895fd473c827cf1822f3bd8249.tar.gz | |
*.[ch] *_INIT macros: use { 0 } for a "zero out" idiom
In C it isn't required to specify that all members of a struct are
zero'd out to 0, NULL or '\0', just providing a "{ 0 }" will
accomplish that.
Let's also change code that provided N zero'd fields to just
provide one, and change e.g. "{ NULL }" to "{ 0 }" for
consistency. I.e. even if the first member is a pointer let's use "0"
instead of "NULL". The point of using "0" consistently is to pick one,
and to not have the reader wonder why we're not using the same pattern
everywhere.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'revision.c')
| -rw-r--r-- | revision.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/revision.c b/revision.c index 0dabb5a0bc..73e5004d60 100644 --- a/revision.c +++ b/revision.c @@ -249,7 +249,7 @@ struct commit_stack { struct commit **items; size_t nr, alloc; }; -#define COMMIT_STACK_INIT { NULL, 0, 0 } +#define COMMIT_STACK_INIT { 0 } static void commit_stack_push(struct commit_stack *stack, struct commit *commit) { |