diff options
author | dim <dim@FreeBSD.org> | 2016-09-07 19:25:08 +0000 |
---|---|---|
committer | dim <dim@FreeBSD.org> | 2016-09-07 19:25:08 +0000 |
commit | be91514db3ffa8ef84f912275d8c612f4a1a8407 (patch) | |
tree | b71eec3c5938e28b090b740a31ad53c14a28f94d /contrib/diff/src/analyze.c | |
parent | 83216cdbb72b65e649ebf7582ea72cb089229b41 (diff) | |
download | FreeBSD-src-be91514db3ffa8ef84f912275d8c612f4a1a8407.zip FreeBSD-src-be91514db3ffa8ef84f912275d8c612f4a1a8407.tar.gz |
MFC r305389:
With clang 3.9.0, compiling sys/netinet/igmp.c results in the following
warning:
sys/netinet/igmp.c:546:21: error: implicit conversion from 'int' to 'char' changes value from 148 to -108 [-Werror,-Wconstant-conversion]
p->ipopt_list[0] = IPOPT_RA; /* Router Alert Option */
~ ^~~~~~~~
sys/netinet/ip.h:153:19: note: expanded from macro 'IPOPT_RA'
#define IPOPT_RA 148 /* router alert */
^~~
This is because ipopt_list is an array of char, so IPOPT_RA is wrapped
to a negative value. It would be nice to change ipopt_list to an array
of u_char, but it changes the signature of the public struct ipoption,
so add an explicit cast to suppress the warning.
Reviewed by: imp
Differential Revision: https://reviews.freebsd.org/D7777
Diffstat (limited to 'contrib/diff/src/analyze.c')
0 files changed, 0 insertions, 0 deletions