On Thu, May 15, 2025 at 01:11:45PM +0000, Johannes Schindelin via GitGitGadget wrote: > From: Johannes Schindelin <johannes.schindelin@xxxxxx> > > While `if (i <= 0) ... else if (i > 0) ...` is technically equivalent to > `if (i <= 0) ... else ...`, the latter is vastly easier to read because > it avoids writing out a condition that is unnecessary. Let's drop such > unnecessary conditions. > > Pointed out by CodeQL. Yeah, I'd agree that it is easier (otherwise, you are left wondering if there is an "else" case you are missing). > help.c | 2 +- > transport-helper.c | 2 +- Both spots look good to me. -Peff
