From: Peijian Ju <eric.peijian@gmail.com>
Date: Tue, 16 Jul 2024 02:59:32 GMT
Subject: Re: [PATCH 5/6] cat-file: add declaration of variable i inside its for loop
Message-ID: <CAN2LT1A2c6M21GtEhnBN_bNYOW7csKOgK9fK4-f1HKPa-Vb28w@mail.gmail.com>
In-Reply-To: <CAOLa=ZRH7=Uj77Wd4WQW=KR8ppKw6nXZRH7pNZpHBPz4HhX8MQ@mail.gmail.com>

On Wed, Jul 10, 2024 at 6:16 AM Karthik Nayak <karthik.188@gmail.com> wrote:
>
> Eric Ju <eric.peijian@gmail.com> writes:
>
> > Some code declares variable i and only uses it
> > in a for loop, not in any other logic outside the loop.
> >
> > Change the declaration of i to be inside the for loop for readability.
> >
>
> If we're doing this anyways, we could replace the 'int' with 'size_t'
> too.
>

Thank you. Fixed in V2

> [snip]