On Tue, Apr 23, 2002 at 05:44:25PM -0400, Paul Jarc wrote: > print-nfs.c: In function `nfs_printfh': > print-nfs.c:761: warning: passing arg 6 of `Parse_fh' from incompatible pointer type > > This can be silenced by changing the type of sfsname to "const char*", > and then changing this line: > *sfsname = 0; > to: > *(char*)sfsname = 0; > (A separate char* variable could also be used for this modification > instead, so no casting would be necessary.)
The current CVS code does the latter. > print-radius.c: In function `print_attr_time': > print-radius.c:664: warning: passing arg 2 of `strlcpy' makes pointer from integer >without a cast > > This can be silenced by "#include <time.h>". The current CVS code does that. > setsignal.c: In function `setsignal': > setsignal.c:72: warning: return makes pointer from integer without a cast > > I'm not sure why it's using sigset at all. I have sigaction, but the > output from ./configure didn't mention any check for it. For some reason, the AC_LBL_TYPE_SIGNAL macro in "aclocal.m4" "prefer[s] sigset() to sigaction()". Nobody on the list has come up with a reason to prefer it, so I'm going to change it to prefer sigaction() and wait until something breaks. > Also, it seems this configure script doesn't notice $CPPFLAGS, which > is inconvenient. What's inconvenient about it? - This is the TCPDUMP workers list. It is archived at http://www.tcpdump.org/lists/workers/index.html To unsubscribe use mailto:[EMAIL PROTECTED]?body=unsubscribe
