<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 16, 2013 at 9:28 PM, Christophe Fergeau <span dir="ltr"><<a href="mailto:cfergeau@redhat.com" target="_blank">cfergeau@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Thu, May 16, 2013 at 05:18:10PM +0200, Marc-André Lureau wrote:<br>
> The widget currently filters out last key press from grab key sequence<br>
> if it's not a modifier key. But this will prevent nested usage of ungrab<br>
> combinations such as shift+f12.<br>
><br>
> <a href="https://bugzilla.redhat.com/show_bug.cgi?id=889962" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=889962</a><br>
><br>
> This will also make the following obsolete:<br>
> <a href="https://bugzilla.redhat.com/show_bug.cgi?id=846005" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=846005</a><br>
<br>
</div>This code was added by<br>
<br>
commit a18782415967fbc036608ccbba6005819ad7f58c<br>
Author: Marc-André Lureau <<a href="mailto:marcandre.lureau@redhat.com">marcandre.lureau@redhat.com</a>><br>
Date:   Thu May 17 23:41:37 2012 +0200<br>
<br>
    If grab sequence is matched, still send modifier keys<br>
<br>
    If the last key pressed from the grab sequence is a modifier<br>
    key, let send it to the guest too.<br>
<br>
    This solves the issue of default grab-sequence being ctrl+alt<br>
    and preventing ctrl+alt+del from working.<br>
<br>
Won't this cause a regression?<br>
<br></blockquote><div><br></div><div>Thanks for checking history, but no, this patch actually removes the filter entirely. The commit you mentionned removed it already partially.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Christophe<br>
<div class="im"><br>
<br>
> ---<br>
>  gtk/spice-widget.c | 5 -----<br>
>  1 file changed, 5 deletions(-)<br>
><br>
> diff --git a/gtk/spice-widget.c b/gtk/spice-widget.c<br>
> index 4f74380..4c67e26 100644<br>
> --- a/gtk/spice-widget.c<br>
> +++ b/gtk/spice-widget.c<br>
> @@ -1321,11 +1321,6 @@ static gboolean key_event(GtkWidget *widget, GdkEventKey *key)<br>
>              else<br>
>                  try_mouse_grab(display);<br>
>          }<br>
> -<br>
> -        // that's the last key pressed from the grab sequence<br>
> -        // let send it to the remote if it's a modifier key<br>
> -        if (!key->is_modifier)<br>
> -            return true;<br>
>      }<br>
><br>
>      if (!d->inputs)<br>
> --<br>
> 1.8.3.rc1.49.g8d97506<br>
><br>
</div>> _______________________________________________<br>
> Spice-devel mailing list<br>
> <a href="mailto:Spice-devel@lists.freedesktop.org">Spice-devel@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/spice-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/spice-devel</a><br>
</blockquote></div><br><br clear="all"><br>-- <br>Marc-André Lureau
</div></div>