summaryrefslogtreecommitdiff
path: root/builtin/diff-index.c
diff options
context:
space:
mode:
authorLibravatar Ævar Arnfjörð Bjarmason <avarab@gmail.com>2021-07-01 12:51:26 +0200
committerLibravatar Junio C Hamano <gitster@pobox.com>2021-07-01 12:32:22 -0700
commit5726a6b4012cd41701927a6637b9f2070e7760ee (patch)
treea89b3b9c32f4399b80f1d2d15214b8d9799f4ec8 /builtin/diff-index.c
parent*.h: move some *_INIT to designated initializers (diff)
downloadtgif-5726a6b4012cd41701927a6637b9f2070e7760ee.tar.xz
*.c *_init(): define in terms of corresponding *_INIT macro
Change the common patter in the codebase of duplicating the initialization logic between an *_INIT macro and a corresponding *_init() function to use the macro as the canonical source of truth. Now we no longer need to keep the function up-to-date with the macro version. This implements a suggestion by Jeff King who found that under -O2 [1] modern compilers will init new version in place without the extra copy[1]. The performance of a single *_init() won't matter in most cases, but even if it does we're going to be producing efficient machine code to perform these operations. 1. https://lore.kernel.org/git/YNyrDxUO1PlGJvCn@coredump.intra.peff.net/ Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/diff-index.c')
0 files changed, 0 insertions, 0 deletions