Re: [PATCH spice-gtk] spice-widget: release mouse grab on keyboard-grab-inhibit

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

 



Hi

On Wed, Jul 4, 2012 at 9:07 AM, Hans de Goede <hdegoede@xxxxxxxxxx> wrote:
> This patch also releases the mouse grab on keyboard-grab-inhibit, fixing
> the policykit dialog not showing (and thus usb redir not working) when
> running with server mouse mode. Note that this makes the inhibit-keyboard-grab
> name of the property no longer really cover what it does, but allas it is
> part of our ABI...

I suggest this kind of API change be handled gracefully when we can:
add an addition property such as "inhibit-grabs", and mark the old one
as deprecated with any of the G_FOO_DEPRECATED macro. The choice of
keeping old behaviour or changing behaviour for the deprecated method
is debatable, and is probably case by case. Since you proposed to
change the behaviour silently, I propose it is changed too for the
deprecated property anyway.

regards

-- 
Marc-André Lureau
_______________________________________________
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]