[Spice-devel] [spice-server 7/9] syntax-check: Add missing <config.h> includes

Frediano Ziglio fziglio at redhat.com
Thu Oct 27 14:57:17 UTC 2016


> 
> ---
>  server/event-loop.c             | 3 +++
>  server/tests/stream-test.c      | 4 ++++
>  server/tests/test-qxl-parsing.c | 3 +++
>  3 files changed, 10 insertions(+)
> 
> diff --git a/server/event-loop.c b/server/event-loop.c
> index 1b57460..80add0b 100644
> --- a/server/event-loop.c
> +++ b/server/event-loop.c
> @@ -21,6 +21,9 @@
>   *
>   * const SpiceCoreInterfaceInternal event_loop_core;
>   */
> +#ifdef HAVE_CONFIG_H
> +#include <config.h>
> +#endif
>  
>  #include "red-common.h"
>  
> diff --git a/server/tests/stream-test.c b/server/tests/stream-test.c
> index 623a394..9c43d74 100644
> --- a/server/tests/stream-test.c
> +++ b/server/tests/stream-test.c
> @@ -1,3 +1,7 @@
> +#ifdef HAVE_CONFIG_H
> +#include <config.h>
> +#endif
> +
>  #include <sys/types.h>
>  #include <sys/socket.h>
>  #include <errno.h>
> diff --git a/server/tests/test-qxl-parsing.c
> b/server/tests/test-qxl-parsing.c
> index a3b2e6c..a8dd27b 100644
> --- a/server/tests/test-qxl-parsing.c
> +++ b/server/tests/test-qxl-parsing.c
> @@ -1,5 +1,8 @@
>  /* Do some tests on memory parsing
>   */
> +#ifdef HAVE_CONFIG_H
> +#include <config.h>
> +#endif
>  
>  #undef NDEBUG
>  #include <string.h>


Acked-by: Frediano Ziglio <fziglio at redhat.com>

Frediano

OT: as config.h is always included does it make sense to check for HAVE_CONFIG_H ??


More information about the Spice-devel mailing list