From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 10 Dec 2020 06:56:19 GMT
Subject: Re: [PATCH 2/7] diffcore-rename: remove unnecessary if-clause
Message-ID: <xmqqpn3iw3n0.fsf@gitster.c.googlers.com>
In-Reply-To: <CABPp-BFcSV_LPy-_AW+c-1OgzAyHfEM7D+YL+fFwxYKDheqxEg@mail.gmail.com>

Elijah Newren <newren@gmail.com> writes:

> Ah, so would you be okay replacing these with
>
>    if (st_mult(num_targets, limited_sources) <= st_mult(rename_limit,
> rename_limit))

It's subtle that the use of size_t is a sensible thing to do, and
definitely deserves an in-line comment, but I think that is a good
way to go.