[Spice-devel] [PATCH spice-gtk] channel: rely on couroutine instead of channel state

Marc-André Lureau marcandre.lureau at gmail.com
Sun Dec 2 06:16:45 PST 2012


On Sun, Dec 2, 2012 at 12:34 PM, Hans de Goede <hdegoede at redhat.com> wrote:

> Hmm, this seems like an unrelated change/fix and one which could do with
> its own
> commit message. Otherwise ack.
>

It's actually related, I removed a case in the current iterate_read() (I
replaced with a pre-condition instead):

+    g_return_if_fail(c->state != SPICE_CHANNEL_STATE_MIGRATING)

-    case SPICE_CHANNEL_STATE_MIGRATING:
-        return;

Without that change, we might continue reading/handling incoming data
during migration.

-- 
Marc-André Lureau
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20121202/544d38bb/attachment.html>


More information about the Spice-devel mailing list