Re: [PATCH v5 11/13] xdiff: rename rchg -> changed in xdfile_t

From: Phillip Wood <phillip.wood123@gmail.com>
Date: Sep 24, 2025, 15:18 UTC
Message-ID: <6057413c-97a7-4b4f-add8-173ec6f74e2b@gmail.com>
In-Reply-To: <CAH=ZcbA+MLbyNp3f7RbMFA6vrDZC6yRKVh7ryVer1Ozg3d4oSg@mail.gmail.com>

On 24/09/2025 16:10, Ezekiel Newren wrote:

Show 14 quoted lines

> On Wed, Sep 24, 2025 at 4:21 AM Phillip Wood <phillip.wood123@gmail.com> wrote:
>>
>> On 23/09/2025 22:24, Ezekiel Newren via GitGitGadget wrote:
>>> From: Ezekiel Newren <ezekielnewren@gmail.com>
>>
>> I agree "changed" is a better name but the commit message should explain
>> what "rchg" is used for so that someone who is not familiar with the
>> code can understand why the change in name is desirable.
> 
> The field rchg (now 'changed') declares if a line in a file is changed
> or not. A later commit will change it's type from 'char' to 'bool'
> to make its purpose even more clear.
> 
> Something like that?

That's great

Phillip