Re: [PATCH spice-server 06/22] reds: use bool for RedsState::is_client_mouse_allowed

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

 



On Fri, Jun 16, 2017 at 10:13:18AM +0100, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  server/main-channel-client.c | 4 ++--
>  server/main-channel-client.h | 2 +-
>  server/main-channel.c        | 2 +-
>  server/main-channel.h        | 2 +-
>  server/reds-private.h        | 2 +-
>  server/reds.c                | 2 +-
>  6 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/server/main-channel-client.c b/server/main-channel-client.c
> index ae8d2d5..f741ed5 100644
> --- a/server/main-channel-client.c
> +++ b/server/main-channel-client.c
> @@ -332,7 +332,7 @@ void main_channel_client_push_agent_data(MainChannelClient *mcc, uint8_t* data,
>  static RedPipeItem *main_init_item_new(int connection_id,
>                                         int display_channels_hint,
>                                         int current_mouse_mode,
> -                                       int is_client_mouse_allowed,
> +                                       bool is_client_mouse_allowed,
>                                         int multi_media_time,
>                                         int ram_hint)

I haven't looked at the rest of the series yet, but this function is
doing
item->is_client_mouse_allowed = is_client_mouse_allowed;
so it might be worth changing the type of
RedInitPipeItem::is_client_mouse_allowed in this patch (or before?).
Not really important if this is split in a subsequent patch.

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]