Re: [PATCH 13/13] object-file: drop support for writing objects with unknown types

From: Junio C Hamano <gitster@pobox.com>
Date: May 19, 2025, 15:32 UTC
Message-ID: <xmqq1pskpqdw.fsf@gitster.g>
In-Reply-To: <20250516045013.GM22242@coredump.intra.peff.net>

Jeff King <peff@peff.net> writes:

Show 13 quoted lines

> Since "hash-object --literally" no longer supports objects with unknown
> types, there are now no callers of write_object_file_literally() and its
> helpers. Let's drop them to simplify the code.
>
> In particular, this gets rid of some ugly copy-and-paste code from
> write_object_file_literally(), which is a parallel implementation of
> write_object_file(). When the split was originally made, the two weren't
> that long, but commits like 63a6745a07 (object-file: update the loose
> object map when writing loose objects, 2023-10-01) ended up having to
> duplicate some tricky code.
>
> This patch drops all of that duplication and should make things less
> error-prone going forward.

Good. Creating broken loose object for the purpose of testing was the only reason to have this "feature", and that is reimplemented in a shell script in the test suite, so this can safely go. Nice.

Thanks.