On Sat, Aug 23, 2008 at 7:57 PM, Arnaud Le Blanc <[EMAIL PROTECTED]> wrote:
> The commit adds AC_C_INLINE in config.m4, which defines "inline" appropriatly
> __inline or __inline__ if the compiler does not know the "inline" keyword.
> Basically it does what you added in your commit, but without forcing inlining,
The latter is an optimization, I should have added it in my commit
msg. By the way, config.m4 is unix only and is completely useless on
> and unlike a test against some known compiler macros, this will work on all
> compilers by checking what they actually support.
Except windows (and certainly other).
> For instance your code will check only GCC and the MS compiler, and the last
> #else may miss the fact that the compiler actually supported __inline or
> For "static" before "inline", it avoids a problem with the Sun compiler
> (without static the compiler references the function but does not actually
> outputs it).
So please apply the correct fix, but keep the windows linux and
windows build safe. That certainly means to add what you need in the
src or add a portable way to use inline (like in zend, which is where
I took the #ifdef). That would be actually the best solution (in
php_config.h and config.w32.h).
>> > +# ifndef HAVE_U_INT32_T
>> > + typedef uint32_t u_int32_t;
>> > +# endif
>> > +#endif
>> I don't understand this part. If HAVE_U_INT32_T is not defined, you
>> still use u_int32_t?
> If HAVE_U_INT32_T is not defined that means that u_int32_t is not defined.
> So I define u_int32_t it to uint32_t. This makes sense for me.
It defines unint32_t not u_int32_t. The define should be HAVE_UINT32_T
if what you test is uint32_t.
http://blog.thepimp.net | http://www.libgd.org
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php