[virt-tools-list] [PATCH virt-viewer] virt-viewer-app: Set hotkeys when app is constructed

Pavel Grunt pgrunt at redhat.com
Mon Mar 30 12:16:11 UTC 2015


Hey,
> 
> Hey,
> 
> For what it's worth, a bit more details in the log about what is
> causing
> the bug/why this commit is fixing it would be useful. From a quick
> look,
> this might be the call to virt_viewer_app_set_enable_accel() which
> will
> be overridden during property settings? I'd need to test to be sure.
> Or maybe I'm missing something obvious here :)
> 
> Christophe
> 

As you said, it is related to virt_viewer_app_set_enable_accel(). I don't know why it was moved to virt_viewer_app_init(). In _init() we don't have the display which can react to the "enable-accel" notify.
However in constructed() we have the display and thanks to it we can change the accel.

I will send v2...

Thanks,
Pavel




More information about the virt-tools-list mailing list