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
> 

Sooner or later before is sent to the network has to be converted to an
integral value. I'll extend.

Frediano
_______________________________________________
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]