Page MenuHomeFreeBSD

Fix various -Wpointer-compare warnings

Authored by jrtc27 on Oct 6 2019, 3:16 PM.



This warning (comparing a pointer against a zero character literal
rather than NULL) has existed since GCC 7.1.0, and was recently added to
Clang trunk. Almost all of these are harmless, except for fwcontrol's
str2node, which needs to both guard against dereferencing a NULL
pointer (though in practice it appears none of the callers will ever
pass one in), as well as ensure it doesn't parse the empty string as
node 0 due to strtol's awkward interface.

Diff Detail

rS FreeBSD src repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

jrtc27 created this revision.Oct 6 2019, 3:16 PM
brooks added a comment.Oct 8 2019, 9:00 PM

I've merged the tcsh changes in via an update to tcsh 6.21.00 in rS353325.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 8 2019, 9:14 PM
This revision was automatically updated to reflect the committed changes.