Re: [dbus PATCH 3/4] Annotate properties for which we will not emit changed signal

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

 



On Thu, 2018-05-03 at 14:46 +0200, Pavel Hrdina wrote:
> For some of these properties there is no libvirt event to detect the
> change and for properties where we could somehow detect the change
> let's annotate them as well.
> 
> We could change the properties to methods but with the annotation we
> can keep them as properties in order to allow to get them by single
> D-Bus call.
> 
> Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
> ---
>  data/org.libvirt.Connect.xml | 3 +++
>  data/org.libvirt.Domain.xml  | 9 +++++++++
>  data/org.libvirt.Network.xml | 3 +++
>  3 files changed, 15 insertions(+)
> 

Maybe it would be nicer to set `false` as the default annotation for
EmitsChangedSignal in the enclosing interface element and explicitly
mark the rest. But I am ok with this as well.

Reviewed-by: Katerina Koukiou <kkoukiou@xxxxxxxxxx>

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux