Subject: Re: [PATCH v2 7/8] packed-backend: check whether the "packed-refs" is sorted

From: Junio C Hamano <[email-masked]>
Date: Thu, 30 Jan 2025 11:02:18 -0800

shejialuo <[email-masked]> writes:

> We will always try to sort the "packed-refs" increasingly by comparing
> the refname. So, we should add checks to verify whether the "packed-refs"
> is sorted.

Do this _ONLY_ when the packed-refs file has a header that declares
"sorted" trait.  Insisting on a packed-refs file that does not would
mean you are stricter than the runtime contract allows.

> +struct fsck_packed_ref_entry {
> +	int line_number;
> +
> +	struct snapshot_record record;
> +};

Not a huge deal, as 1 billion is still plenty of a large number, but
the same comment on the line-number applies here.  We might want to
consistently use ulong for line numbers of files we read from.