Re: [PATCH v6 4/4] hook: introduce the receive-report hook

From: Karthik Nayak <karthik.188@gmail.com>
Date: Sep 3, 2026, 16:32 UTC
Message-ID: <CAOLa=ZR30-=dBrayaWMZAv6Zm=u3w_Va1vNcPGBMmXpVk3V1Ag@mail.gmail.com>
In-Reply-To: <aplGBSWKfn02k7Ku@pks.im>

Patrick Steinhardt <ps@pks.im> writes:

Show 19 quoted lines

> On Thu, Sep 03, 2026 at 11:28:01AM +0200, Karthik Nayak wrote:
>> diff --git a/builtin/receive-pack.c b/builtin/receive-pack.c
>> index 9ac10465ac..edfd5cf9dc 100644
>> --- a/builtin/receive-pack.c
>> +++ b/builtin/receive-pack.c
>> @@ -2535,6 +2570,13 @@ static void update_shallow_info(struct command *commands,
>>  	free(ref_status);
>>  }
>>
>> +static void override_cmds_error(struct command *commands, const char *err)
>> +{
>> +	for (struct command *cmd = commands; cmd; cmd = cmd->next) {
>> +		cmd->error_string = err;
>> +	}
>> +}
>
> Micronit: unnecessary curly braces.
>
> Patrick

Yup, will remove. Will send in a new version tomorrow, if there are no other reviews.