On Thu, Mar 09, 2017 at 09:16:51AM -0500, Frediano Ziglio wrote: > > > > --- > > server/red-channel-client.c | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > diff --git a/server/red-channel-client.c b/server/red-channel-client.c > > index 63877ab..47583c7 100644 > > --- a/server/red-channel-client.c > > +++ b/server/red-channel-client.c > > @@ -133,15 +133,15 @@ struct RedChannelClientPrivate > > } urgent; > > } send_data; > > > > - int during_send; > > + gboolean during_send; > > GQueue pipe; > > > > RedChannelCapabilities remote_caps; > > - int is_mini_header; > > + gboolean is_mini_header; > > gboolean destroying; > > > > - int wait_migrate_data; > > - int wait_migrate_flush_mark; > > + gboolean wait_migrate_data; > > + gboolean wait_migrate_flush_mark; > > > > RedChannelClientLatencyMonitor latency_monitor; > > RedChannelClientConnectivityMonitor connectivity_monitor; > > I would stick to bool even if properties are used.
Ah correct, this one just aimed to keep consistency with what is already being done (see 'gboolean destroying'), this was before the bool vs gboolean discussion I think. I can change it. Christophe
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/spice-devel
