From: Andrei Rybak <rybak.a.v@gmail.com>
Date: Fri, 27 May 2022 22:53:00 GMT
Subject: Re: [PATCH 4/5] receive-pack: use bug() and BUG_if_bug()
Message-ID: <55585ede-0135-b996-08da-fc1919827981@gmail.com>
In-Reply-To: <patch-4.5-c590f4273c0-20220521T170939Z-avarab@gmail.com>

On 21/05/2022 19:14, Ævar Arnfjörð Bjarmason wrote:
> Amend code added in a6a84319686 (receive-pack.c: shorten the
> execute_commands loop over all commands, 2015-01-07) and amended to
> hard die in b6a4788586d (receive-pack.c: die instead of error in case
> of possible future bug, 2015-01-07) to the new bug() function instead.

A verb is missing after "to". "to use the new bug() function", perhaps?

> 
> Let's also rename the warn_if_*() function that code is in to
> BUG_if_*(), its name became outdated in b6a4788586d.
> 
> Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
> ---