> 
> test-marshallers.h is missing a #define _H_TEST_MARSHALLERS in order to
> prevent multiple #include for the same header.
> 
> Signed-off-by: Christophe Fergeau <cferg...@redhat.com>
> ---
>  tests/test-marshallers.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/test-marshallers.h b/tests/test-marshallers.h
> index 4eab90f..5b113b7 100644
> --- a/tests/test-marshallers.h
> +++ b/tests/test-marshallers.h
> @@ -1,6 +1,7 @@
>  #include <stdint.h>
>  

OT: why this is outside the guard?

>  #ifndef _H_TEST_MARSHALLERS
> +#define _H_TEST_MARSHALLERS
>  

OT 2: Why not H_SPICE_COMMON_ prefix?

>  typedef struct {
>      uint32_t data_size;

beside, 

acked

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to