On 25/07/28 03:08PM, Patrick Steinhardt wrote:
> diff --git a/refs.c b/refs.c
> index fd9a5f36b20..b820c3908bd 100644
> --- a/refs.c
> +++ b/refs.c
> @@ -2942,6 +2942,7 @@ struct migration_data {
> struct ref_transaction *transaction;
> struct strbuf *errbuf;
> struct strbuf sb, name, mail;
> + uint64_t index;
> };
>
> static int migrate_one_ref(const char *refname, const char *referent UNUSED, const struct object_id *oid,
> @@ -2974,14 +2975,6 @@ static int migrate_one_ref(const char *refname, const char *referent UNUSED, con
> return ret;
> }
>
> -struct reflog_migration_data {
> - uint64_t index;
> - struct ref_store *old_refs;
> - struct ref_transaction *transaction;
> - struct strbuf *errbuf;
> - struct strbuf *sb, *name, *mail;
> -};
When I recently was looking at this in [1], I remember finding this
rather awkward/confusing. Very happy to see it go now in favor of
something much simpler :)
-Justin
[1]: <tg72v5vgu56b6akawy7sfapi2qtrmy7q3uruhersy4dtzkpvju@wamlylndp3xv>
