From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 19 Dec 2024 10:35:13 GMT
Subject: Re: [PATCH v6] git: use logical-not operator to toggle between 0 and 1
Message-ID: <xmqqttb0m04e.fsf@gitster.g>
In-Reply-To: <pull.1620.v6.git.git.1734541037465.gitgitgadget@gmail.com>

"AreaZR via GitGitGadget" <gitgitgadget@gmail.com> writes:

> From: Seija Kijin <doremylover123@gmail.com>
>
> If it is known that an int is either 1 or 0,

This justification needs to be updated.

The point of using flip = !flip pattern is not that you do not
assume "1 or 0", but follow "zero or non-zero", which is more
idiomatic in C.

Other than that, 

>       -				flipped_block = (flipped_block + 1) % 2;
>      -+				flipped_block ^= 1;
>      ++				flipped_block = !flipped_block;

it indeed is a good change to avoid modulo-2 like this.

Thanks.