]> git.ipfire.org Git - thirdparty/git.git/commit - alloc.c
alloc: write out allocator definitions
authorJeff King <peff@peff.net>
Sun, 13 Jul 2014 06:41:51 +0000 (02:41 -0400)
committerJunio C Hamano <gitster@pobox.com>
Mon, 14 Jul 2014 01:59:04 +0000 (18:59 -0700)
commit600e2a69df6dfde134aba0ca88cfe1bd2c88ecbb
treea57376d649a5239e801b8bb4426c91214735e10c
parent225ea22046a1193bd934a8ea308fa4a7788c9796
alloc: write out allocator definitions

Because the allocator functions for tree, blobs, etc are all
very similar, we originally used a macro to avoid repeating
ourselves. Since the prior commit, though, the heavy lifting
is done by an inline helper function.  The macro does still
save us a few lines, but at some readability cost.  It
obfuscates the function definitions (and makes them hard to
find via grep).

Much worse, though, is the fact that it isn't used
consistently for all allocators. Somebody coming later may
be tempted to modify DEFINE_ALLOCATOR, but they would miss
alloc_commit_node, which is treated specially.

Let's just drop the macro and write everything out
explicitly.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
alloc.c