Re: [Spice-devel] [PATCH 5/7] Don't increment num_clients_mig_wait twice

2016-10-13 Thread Uri Lublin
On 10/12/2016 05:19 PM, Pavel Grunt wrote: On Wed, 2016-10-12 at 10:13 -0400, Frediano Ziglio wrote: On Wed, 2016-10-12 at 06:57 -0400, Frediano Ziglio wrote: When MainChannelClient was split to a separate file, the responsibility for incrementing this field was supposed to belong to the Mai

Re: [Spice-devel] [PATCH 5/7] Don't increment num_clients_mig_wait twice

2016-10-12 Thread Pavel Grunt
On Wed, 2016-10-12 at 10:13 -0400, Frediano Ziglio wrote: > > > > On Wed, 2016-10-12 at 06:57 -0400, Frediano Ziglio wrote: > > > > > > > > > > > > When MainChannelClient was split to a separate file, the > > > > responsibility > > > > for incrementing this field was supposed to belong to the >

Re: [Spice-devel] [PATCH 5/7] Don't increment num_clients_mig_wait twice

2016-10-12 Thread Frediano Ziglio
> > On Wed, 2016-10-12 at 06:57 -0400, Frediano Ziglio wrote: > > > > > > > > > When MainChannelClient was split to a separate file, the > > > responsibility > > > for incrementing this field was supposed to belong to the > > > MainChannel > > > function (main_channel_connect_semi_seamless()), b

Re: [Spice-devel] [PATCH 5/7] Don't increment num_clients_mig_wait twice

2016-10-12 Thread Jonathon Jongsma
On Wed, 2016-10-12 at 06:57 -0400, Frediano Ziglio wrote: > > > > > > When MainChannelClient was split to a separate file, the > > responsibility > > for incrementing this field was supposed to belong to the > > MainChannel > > function (main_channel_connect_semi_seamless()), but by mistake it >

Re: [Spice-devel] [PATCH 5/7] Don't increment num_clients_mig_wait twice

2016-10-12 Thread Frediano Ziglio
> > When MainChannelClient was split to a separate file, the responsibility > for incrementing this field was supposed to belong to the MainChannel > function (main_channel_connect_semi_seamless()), but by mistake it was > incremented both there and in the client function > (main_channel_client_co

[Spice-devel] [PATCH 5/7] Don't increment num_clients_mig_wait twice

2016-10-11 Thread Jonathon Jongsma
When MainChannelClient was split to a separate file, the responsibility for incrementing this field was supposed to belong to the MainChannel function (main_channel_connect_semi_seamless()), but by mistake it was incremented both there and in the client function (main_channel_client_connect_semi_se