Re: [PATCH] Remove duplicate typedef from headers

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

 



On Wed, Mar 30, 2016 at 05:33:50PM +0100, Frediano Ziglio wrote:
> This is causing compile errors with older gcc versions.
> 
> reds.h includes reds-stream.h through red-channel.h so it will get the
> RedsState typedef this way.
> 
> red-qxl.h includes spice-qxl.h through red-channel.h and then spice.h,
> so it will get the QXLState typedef this way.
> 
> Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  server/main-dispatcher.h | 2 +-
>  server/red-qxl.h         | 2 --
>  server/reds-stream.h     | 3 +--
>  3 files changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/server/main-dispatcher.h b/server/main-dispatcher.h
> index ffe82c8..35811d7 100644
> --- a/server/main-dispatcher.h
> +++ b/server/main-dispatcher.h
> @@ -58,6 +58,6 @@ void main_dispatcher_set_mm_time_latency(MainDispatcher *self, RedClient *client
>   */
>  void main_dispatcher_client_disconnect(MainDispatcher *self, RedClient *client);
>  
> -MainDispatcher* main_dispatcher_new(RedsState *reds, SpiceCoreInterfaceInternal *core);
> +MainDispatcher* main_dispatcher_new(SpiceServer *reds, SpiceCoreInterfaceInternal *core);

With the .c file still using RedsState * for main_dispatcher_new(), NACK.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://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]