Hi, > +#include <stdint.h> > +#include <stdio.h> > +#include <string.h> > +#include <stdlib.h> > +#include <unistd.h> > +#include <stdlib.h> > +#include <malloc.h>
I don't think either unistd.h or malloc.h needs to be included here. > +void __attribute__((noinline)) __attribute__((noinline)) is GCC specific, so it needs to be defined in pixman-compiler.h similarly to force_inline. I think the Visual C version is called __declspec(noinline). Also, in the gettime patch, utils.c needs to include <sys/time.h> to get the prototype for gettimeofday(). It should be checked for in configure.ac with AC_CHECK_HEADER(). Other than that, these patches look good to me. Thanks, Soren _______________________________________________ Pixman mailing list Pixman@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/pixman