If struct _zend_arg_info is an incomplete type, the declarations in
zend_modules.h like:
extern struct _zend_arg_info first_arg_force_ref[2];
are not valid C code (it doesn't make sense to declare an array if the
compiler does not know what size each element is, so this is not hard to
understand).
I guess this is what the Netware guys already found and why they added
the conditional include of zend_compile.h to declare the zend_arg_info
struct. GCC 4 will also reject this code. Any reason why this include
cannot be done unconditionally?
FYI, there are about 3000 lines of warnings in HEAD with a current GCC 4
snapshot compiled at -Wall:
http://www.apache.org/~jorton/gcc4.php-Wall.log
--- Zend/zend_modules.h 30 Oct 2004 19:27:42 -0000 1.62
+++ Zend/zend_modules.h 17 Feb 2005 16:31:21 -0000
@@ -23,10 +23,8 @@
#define MODULES_H
#include "zend.h"
-
-#ifdef NETWARE
#include "zend_compile.h"
-#endif
+
#define INIT_FUNC_ARGS int type, int module_number TSRMLS_DC
#define INIT_FUNC_ARGS_PASSTHRU type, module_number TSRMLS_CC
#define SHUTDOWN_FUNC_ARGS int type, int module_number TSRMLS_DC
