<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 12, 2014 at 7:52 PM, Zeeshan Ali (Khattak) <span dir="ltr"><<a href="mailto:zeeshanak@gnome.org" target="_blank">zeeshanak@gnome.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Sep 10, 2014 at 3:52 PM, Zeeshan Ali (Khattak)<br>
<<a href="mailto:zeeshanak@gnome.org">zeeshanak@gnome.org</a>> wrote:<br>
> On Wed, Sep 10, 2014 at 3:35 PM, Christophe Fergeau <<a href="mailto:cfergeau@redhat.com">cfergeau@redhat.com</a>> wrote:<br>
>> Since commit 9cf9ca434, spice_channel_iterate() will report a<br>
>> SPICE_CHANNEL_ERROR_IO error to library users when<br>
>> SpiceChannel::has_error is set. In particular, when the server side<br>
>> closes its SPICE sockets because the VM is being shut down, an IO error<br>
>> will get reported. Prior to this change, a channel-closed event was<br>
>> reported on graceful VM shutdowns as there was<br>
>> a g_socket_condition_check() guarding the emission of the IO error<br>
>> signal.<br>
>><br>
>> This commit readds the g_socket_condition_check() test, but only when<br>
>> SpiceChannel::has_error is set.<br>
>><br>
>> This fixes <a href="https://bugs.freedesktop.org/show_bug.cgi?id=83692" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=83692</a><br>
>> ---<br>
><br>
> Just wanted to say that I tested this patch and it fixes the bug above.<br>
<br>
</span>Anyone up for review? It would be nice to get this in and (if<br>
possible) even released before GNOME's code freeze break on Wednesday.<br>
Won't be bad if release before 3.14 release in two weeks either.<br></blockquote><div><br></div><div><br></div><div>I have considered your previous comment as an ACK.</div><div>Anyway, ACK!</div><div><br></div><div>Best Regards,</div></div>-- <br><div>Fabiano Fidêncio</div>
</div></div>