<div dir="ltr"><div><div>Hi Pavel,<br><br></div></div><div class="gmail_extra"><div class="gmail_quote">On Wed, Dec 16, 2015 at 3:01 PM, Pavel Grunt <span dir="ltr"><<a href="mailto:pgrunt@redhat.com" target="_blank">pgrunt@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Lukas,<br>
<br>
is the client disconnected?<br></blockquote><div><br>If the function that handles preferred compression returns FALSE, red_channel_client_disconnect() is called.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class=""><br>
On Wed, 2015-12-16 at 14:48 +0100, Lukas Venhoda wrote:<br>
> Don't diconnect client, if lz4 is requested as a preferred<br>
</span>typo     ^<br>
<span class="">> compression,<br>
> but the server doesn't support it.<br>
<br>
</span>Please add the reference to the bug <br>
<span class=""><br>
> ---<br>
> Sidenote:<br>
> Should server disconnect, when unsupported compression is requested?<br>
> FE if client sends unknown compression, and this switch goes to<br>
> default.<br>
> Default could always return TRUE, just not set the compression.<br>
</span>I agree with you, in this case (it is just a preferred compression) it<br>
make sense to return TRUE.<br>
<br>
Pavel<a href="http://lists.freedesktop.org/mailman/listinfo/spice-devel" rel="noreferrer" target="_blank"></a><br>
</blockquote></div><br>I'll add reference, fix the typo and change default in the switch statement, to return TRUE, that should be a better fix.<br clear="all"><br>-- <br><div class="gmail_signature">Lukas Venhoda</div>
</div></div>