Hi,

Recently, I imported imsg.c from OpenBSD to the
FreeBSD base system's libopenbsd:

https://svnweb.freebsd.org/changeset/base/290375

When compiling on FreeBSD, we get a compiler warning with clang:

cc  -O2 -pipe   -I/opt2/branches/head2/lib/libopenbsd -std=gnu99
-fstack-protector-strong -Wsystem-headers -Wall -Wno-format-y2k -W
-Wno-unused-parameter -Wstrict-prototypes -Wmissing-prototypes
-Wpointer-arith -Wno-uninitialized -Wno-pointer-sign -Wno-empty-body
-Wno-string-plus-int -Wno-unused-const-variable -Wno-tautological-compare
-Wno-unused-value -Wno-parentheses-equality -Wno-unused-function
-Wno-enum-conversion -Wno-unused-local-typedef -Qunused-arguments -c
/opt2/branches/head2/lib/libopenbsd/imsg.c -o imsg.o
/opt2/branches/head2/lib/libopenbsd/imsg.c:78:6: warning: comparison of
integers of different signs: 'unsigned long' and 'int' [-Wsign-compare]
            >= getdtablesize()) {
            ^  ~~~~~~~~~~~~~~~
1 warning generated.

I can certainly patch the code with a cast on FreeBSD to get rid of the
compiler
warning.  However, I was wondering if there is a good fix that we can share
between
OpenBSD and FreeBSD?

Thanks.
--
Craig

Reply via email to