[Spice-devel] [PATCH spice-server] display-channel: Move _Drawable declaration to private header

[Jonathon Jongsma]

Sure

Acked-by: Jonathon Jongsma <jjongsma at redhat.com>


On Wed, 2017-02-01 at 13:41 +0000, Frediano Ziglio wrote:
> The structure is used only to allocate private data.
> 
> Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> ---
>  server/display-channel-private.h | 9 +++++++++
>  server/display-channel.h         | 9 ---------
>  2 files changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/server/display-channel-private.h b/server/display-
> channel-private.h
> index a930cce..afb28b6 100644
> --- a/server/display-channel-private.h
> +++ b/server/display-channel-private.h
> @@ -20,6 +20,15 @@
>  
>  #include "display-channel.h"
>  
> +#define NUM_DRAWABLES 1000
> +typedef struct _Drawable _Drawable;
> +struct _Drawable {
> +    union {
> +        Drawable drawable;
> +        _Drawable *next;
> +    } u;
> +};
> +
>  struct DisplayChannelPrivate
>  {
>      DisplayChannel *pub;
> diff --git a/server/display-channel.h b/server/display-channel.h
> index 5ecc5c3..497f911 100644
> --- a/server/display-channel.h
> +++ b/server/display-channel.h
> @@ -173,15 +173,6 @@ typedef struct RedSurface {
>      QXLReleaseInfoExt create, destroy;
>  } RedSurface;
>  
> -#define NUM_DRAWABLES 1000
> -typedef struct _Drawable _Drawable;
> -struct _Drawable {
> -    union {
> -        Drawable drawable;
> -        _Drawable *next;
> -    } u;
> -};
> -
>  #define FOREACH_DCC(_channel, _iter, _data) \
>      GLIST_FOREACH((_channel ?
> red_channel_get_clients(RED_CHANNEL(_channel)) : NULL), \
>                    _iter, DisplayChannelClient, _data)