On Sat, 24 Jul 2010, Stefan Farfeleder wrote:
declaring enums like this is not standard C code (seems to be a GCC extension). I don't think we should use this feature in our headers.
This is unfortunate. This is because the size of an enum variable depends on its complete declaration. This is an error unconditionaly with TenDRA. It takes -pedantic to get a warning from gcc.
Modified: head/sys/sys/imgact.h ============================================================================== --- head/sys/sys/imgact.h Sat Jul 24 18:05:10 2010 (r210450) +++ head/sys/sys/imgact.h Sat Jul 24 18:14:34 2010 (r210451) @@ -32,8 +32,6 @@ #ifndef _SYS_IMGACT_H_ #define _SYS_IMGACT_H_ -#include <sys/uio.h> -
The rule that new pollution may not be added to old headers should have prevented the need for this removal.
#define MAXSHELLCMDLEN PAGE_SIZE struct image_args { @@ -75,6 +73,7 @@ struct image_params { #ifdef _KERNEL struct sysentvec; struct thread; +enum uio_seg; #define IMGACT_CORE_COMPRESS 0x01
Bruce _______________________________________________ svn-src-head@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
