From: Junio C Hamano <gitster@pobox.com>
Date: Wed, 03 Sep 2025 05:25:44 GMT
Subject: Re: [GSoC][PATCH 1/5] builtin/pack-refs: factor out core logic into a helper
Message-ID: <xmqq8qiwgl9j.fsf@gitster.g>
In-Reply-To: <aLbEWSHUotkmSiCq@pks.im>

Patrick Steinhardt <ps@pks.im> writes:

>>  builtin/pack-refs.c | 31 ++++++++++++++++++++-----------
>>  pack-refs.h         | 22 ++++++++++++++++++++++
>>  2 files changed, 42 insertions(+), 11 deletions(-)
>>  create mode 100644 pack-refs.h
>
> Shouldn't that header live in "builtin/pack-refs.h"? Makes it way more
> obvious that it exposes functions from "builtin/pack-refs.c".

There shouldn't be any *.h files in builtin/.  since the top-level
is where the library-ish reusable things live, so that they can
eventually be used by more than one *.c files in builtin/ and also
by other *.c files outside builtin.