Re: [PATCH spice-gtk 09/13] migration: remove unnecessary reference

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

 



On Wed, Nov 12, 2014 at 05:03:37PM +0100, Marc-André Lureau wrote:
> On Wed, Nov 12, 2014 at 4:14 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx>
> wrote:
> 
> > Hey, this one can be squashed in patch 04/13
> 
> 
> I prefer to keep this seperate to ease bisection if it introduces issues.
> Furthermore, it's not directly related to 4/13.

Yup, I was about to reply to myself that 4/13 is not the correct one,
but 8/13 is. Got confused about session->priv->migration first moving
from spice_session_set_migration to migrate_connect in 4/13, and then to
main_migrate_connect in 8/13

However, as 8/13 is mostly code movement, I agree that it's better to
keep it this way, and have the removal as an additional patch, so ACK to
this one, and disregard my comments about 4/13.

Christophe

> 
> 
> -- 
> Marc-André Lureau

> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel

Attachment: pgpCwzZzjoMsA.pgp
Description: PGP signature

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