[Pixman] [PATCH 1/1] Add a lowlevel blitter benchmark

Soeren Sandmann sandmann at daimi.au.dk
Fri Sep 17 04:31:08 PDT 2010


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


More information about the Pixman mailing list