PROPOSED FIX to remove warning in 1.1.5-3 sys/types.h

Christian Jullien eligis@wanadoo.fr
Fri Nov 3 11:30:00 GMT 2000


Hi,

The following macro in sys/types.h

#  define	FD_ZERO(p)	(__extension__ (void)({ \
     int i; \
     char *__tmp = (char *)p; \
     for (i = 0; i < sizeof (*(p)); ++i) \
       *__tmp++ = 0; \
}))

raise a new warning 'signed unsigned comparison'
I suggest to replace
int i;
with unsigned int i;

A better solution should be to include stddef (for size_t) and have:
size_t i;
which is the type returned by sizeof.

But, unless I'm totaly wrong, the fastest solution is to use memset that
will be expanded inline with assembly code. Right?

regards.

Christian Jullien


--
Want to unsubscribe from this list?
Send a message to cygwin-unsubscribe@sourceware.cygnus.com



More information about the Cygwin mailing list