On Thu, Apr 9, 2015 at 12:09 AM, Fabiano Fidêncio <fabiano@xxxxxxxxxxxx> wrote:
> On Wed, Apr 8, 2015 at 11:41 PM, Jonathon Jongsma <jjongsma@xxxxxxxxxx> wrote:
>> Freeze notifications before setting the show hints to prevent multiple
>> notification signals from being emitted.
>> ---
>> src/virt-viewer-display.c | 5 ++++-
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/virt-viewer-display.c b/src/virt-viewer-display.c
>> index 66429f1..2b30567 100644
>> --- a/src/virt-viewer-display.c
>> +++ b/src/virt-viewer-display.c
>> @@ -686,9 +686,12 @@ void virt_viewer_display_set_enabled(VirtViewerDisplay *self, gboolean enabled)
>> {
>> g_return_if_fail(VIRT_VIEWER_IS_DISPLAY(self));
>>
>> - virt_viewer_display_set_show_hint(self, VIRT_VIEWER_DISPLAY_SHOW_HINT_SET, TRUE);
>> + g_object_freeze_notify(G_OBJECT(self));
>>
>> + virt_viewer_display_set_show_hint(self, VIRT_VIEWER_DISPLAY_SHOW_HINT_SET, TRUE);
>> virt_viewer_display_set_show_hint(self, VIRT_VIEWER_DISPLAY_SHOW_HINT_DISABLED, !enabled);
>> +
>> + g_object_thaw_notify(G_OBJECT(self));
>> }
>>
>> gboolean virt_viewer_display_get_enabled(VirtViewerDisplay *self)
>> --
>> 2.1.0
>>
>> _______________________________________________
>> virt-tools-list mailing list
>> virt-tools-list@xxxxxxxxxx
>> https://www.redhat.com/mailman/listinfo/virt-tools-list
>
> ACK!
>
> Thanks a lot for these 2 clean ups.
> --
> Fabiano Fidêncio
Pushed, thanks!
--
Fabiano Fidêncio
_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list
