Re: [PATCH spice-gtk 2/2] Don't include super-header from channel-main.h

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

 



Hi

----- Original Message -----
> spice-client.h is a super-header that includes all of the other public
> headers (including channel-main.h). By including this from
> channel-main.h, it creates a circular dependency. This was causing some
> build problems with spice-audio.h:
> 
> In file included from ../../src/spice-client.h:50:0,
>                  from ../../src/channel-main.h:21,
>                  from ../../src/channel-main.c:25:
> ../../src/spice-audio.h:74:44: error: unknown type name 'SpiceMainChannel'
>                                             SpiceMainChannel *main_channel,
>                                             ^
> 

How did you get this error? after your patch or before?

> Instead of including this super-header, just include the one we need:
> spice-channel.h
> ---
>  src/channel-main.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/channel-main.h b/src/channel-main.h
> index 86bb46b..92875dc 100644
> --- a/src/channel-main.h
> +++ b/src/channel-main.h
> @@ -18,7 +18,7 @@
>  #ifndef __SPICE_CLIENT_MAIN_CHANNEL_H__
>  #define __SPICE_CLIENT_MAIN_CHANNEL_H__
>  
> -#include "spice-client.h"
> +#include "spice-channel.h"
>  
>  G_BEGIN_DECLS
>  
> --
> 2.1.0
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
> 
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




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