Re: [spice-common] Remove config.h from header files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/4/18 1:17 PM, Christophe Fergeau wrote:
> This should only be included from c files.
> 
> Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> ---
>  common/client_marshallers.h | 4 ----
>  common/mem.h                | 4 ----
>  common/messages.h           | 4 ----
>  3 files changed, 12 deletions(-)
> 
> diff --git a/common/client_marshallers.h b/common/client_marshallers.h
> index 919e889..912494b 100644
> --- a/common/client_marshallers.h
> +++ b/common/client_marshallers.h
> @@ -19,10 +19,6 @@
>  #ifndef _H_MARSHALLERS
>  #define _H_MARSHALLERS
>  
> -#ifdef HAVE_CONFIG_H
> -# include "config.h"
> -#endif
> -
>  #include <spice/protocol.h>
>  
>  #include "common/generated_client_marshallers.h"
> diff --git a/common/mem.h b/common/mem.h
> index 1c7cfa3..68e024a 100644
> --- a/common/mem.h
> +++ b/common/mem.h
> @@ -23,10 +23,6 @@
>  #include <stdlib.h>
>  #include <spice/macros.h>
>  
> -#ifdef HAVE_CONFIG_H
> -# include <config.h>
> -#endif
> -
>  SPICE_BEGIN_DECLS
>  
>  #ifdef STDC_HEADERS
> diff --git a/common/messages.h b/common/messages.h
> index 55de76e..14d5c64 100644
> --- a/common/messages.h
> +++ b/common/messages.h
> @@ -31,10 +31,6 @@
>  #ifndef _H_MESSAGES
>  #define _H_MESSAGES
>  
> -#ifdef HAVE_CONFIG_H
> -#include "config.h"
> -#endif
> -
>  #include <spice/protocol.h>
>  #include <spice/macros.h>
>  
> 
Acked-by: Eduardo Lima (Etrunko) <etrunko@xxxxxxxxxx>

-- 
Eduardo de Barros Lima (Etrunko)
Software Engineer - RedHat
etrunko@xxxxxxxxxx
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]