[virt-tools-list] [PATCH virt-viewer 1/2] Emit one show-hint notification when enabling a display

[Fabiano Fidêncio]

On Thu, Apr 9, 2015 at 12:09 AM, Fabiano Fidêncio <fabiano at fidencio.org> wrote:
> On Wed, Apr 8, 2015 at 11:41 PM, Jonathon Jongsma <jjongsma at redhat.com> 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 at redhat.com
>> 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