diff options
author | Jeff King <peff@peff.net> | 2014-07-13 02:41:51 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-07-28 10:14:33 -0700 |
commit | 52604d71443315c82d3a5eb008d601daf8bad05b (patch) | |
tree | 5cba23293c43989826ced77abab7ebd7996a21f3 /commit.c | |
parent | 8c3f3f28cb8c946867365292abfc2c33a5707d3e (diff) | |
download | git-52604d71443315c82d3a5eb008d601daf8bad05b.tar.gz |
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>
Diffstat (limited to 'commit.c')
0 files changed, 0 insertions, 0 deletions