diff options
author | Jeff King <peff@peff.net> | 2014-08-26 06:24:20 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-08-27 07:44:27 -0700 |
commit | 2e3dfb216991974b60fdb1933eb3331e03383e61 (patch) | |
tree | a5b7fc4e71e77832dcbb8c5d5a55de9222ac7972 /commit.h | |
parent | 2608c24940c80bf379937e4cefee75e2db79e008 (diff) | |
download | git-2e3dfb216991974b60fdb1933eb3331e03383e61.tar.gz |
log-tree: use FLEX_ARRAY in name_decorationjk/name-decoration-alloc
We are already using the flex-array technique; let's
annotate it with our usual FLEX_ARRAY macro. Besides being
more readable, this is slightly more efficient on compilers
that understand flex-arrays.
Note that we need to bump the allocation in add_name_decoration,
which did not explicitly add one byte for the NUL terminator
of the string we are putting into the flex-array (it did not
need to before, because the struct itself was over-allocated
by one byte).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'commit.h')
-rw-r--r-- | commit.h | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -30,7 +30,7 @@ extern const char *commit_type; struct name_decoration { struct name_decoration *next; int type; - char name[1]; + char name[FLEX_ARRAY]; }; enum decoration_type { |