Re: PATCH: overflow check in isreallycom
Mikael Magnusson <[email protected]>
| Newsgroups | gmane.comp.shells.zsh.devel |
|---|---|
| Message-ID | <CAHYJk3Q7u3=75Qoa5Gz0XswCp1+ew_JD_VLWYPu9C9wg-iP_Ug@mail.gmail.com> |
Ack, I know I caught a situation exactly like this in another change. Thanks for spotting it. It's probably a good idea to add -Wall to my CFLAGS. :) On Mon, May 25, 2026 at 11:48 PM Bart Schaefer <[email protected]> wrote: > > There's an ambiguous if/else chain in exec.c: > > exec.c:1012:5: warning: add explicit braces to avoid dangling else > [-Wdangling-else] > 1012 | else if (!cn->u.name) > | ^ > 1 warning generated. -- Mikael Magnusson