freshlybakedca.ke / git
Git fork
fork atom

pack-objects: swap 'show_{object,commit}_pack_hint'

show_commit_pack_hint() has heretofore been a noop, so its position
within its compilation unit only needs to appear before its first use.

But the following commit will sometimes have `show_commit_pack_hint()`
call `show_object_pack_hint()`, so reorder the former to appear after
the latter to minimize the code movement in that patch.

Suggested-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

authored by

Taylor Blau and committed by
Junio C Hamano 63195f01 8ed5d87b

+6 -6
unified split
+6 -6
builtin/pack-objects.c
··· 3748 return 0; 3749 } 3750 3751 - static void show_commit_pack_hint(struct commit *commit UNUSED, 3752 - void *data UNUSED) 3753 - { 3754 - /* nothing to do; commits don't have a namehash */ 3755 - } 3756 - 3757 static void show_object_pack_hint(struct object *object, const char *name, 3758 void *data UNUSED) 3759 { ··· 3774 oe->no_try_delta = name && no_try_delta(name); 3775 3776 stdin_packs_hints_nr++; 3777 } 3778 3779 static int pack_mtime_cmp(const void *_a, const void *_b)
··· 3748 return 0; 3749 } 3750 3751 static void show_object_pack_hint(struct object *object, const char *name, 3752 void *data UNUSED) 3753 { ··· 3768 oe->no_try_delta = name && no_try_delta(name); 3769 3770 stdin_packs_hints_nr++; 3771 + } 3772 + 3773 + static void show_commit_pack_hint(struct commit *commit UNUSED, 3774 + void *data UNUSED) 3775 + { 3776 + /* nothing to do; commits don't have a namehash */ 3777 } 3778 3779 static int pack_mtime_cmp(const void *_a, const void *_b)