Eric Sunshine <sunshine@xxxxxxxxxxxxxx> writes: > On Sat, Nov 4, 2023 at 1:12 AM Junio C Hamano <gitster@xxxxxxxxx> wrote: >> > + if ((magic & pathspec_magic[i].bit) && (pathspec_magic[i].mnemonic != '\0')) { >> >> Overly long, with a stylo. >> >> Here is what I ended up queuing. >> >> + for (int i = 0; i < ARRAY_SIZE(pathspec_magic); i++) { >> + if ((magic & pathspec_magic[i].bit) && >> + (pathspec_magic[i].mnemonic)) { > > Nit: The no-value-added parentheses around > `pathspec_magic[i].mnemonic` can also be dropped. Yeah, thanks for spotting.