<div dir="ltr"><div><div>Hi! Sorry for the delay, I had some exams. <br><br></div>_set_listen_list () is the only other name that comes into my mind.<br><br>I also think _set_listens () is a bit odd, but  _set_listen_nodes () doesn't sound that odd to me (maybe a bit too technical).<br><br></div><div>I think you should choose the name from those 3, because you have more experience, and I'll update and upload the remaining two patches right away.<br></div><div><br><br></div><div>All the best,<br></div><div>Alex<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 2, 2016 at 5:17 PM, Christophe Fergeau <span dir="ltr"><<a href="mailto:cfergeau@redhat.com" target="_blank">cfergeau@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, Aug 17, 2016 at 06:58:49PM +0300, Visarion Alexandru wrote:<br>
> From: Visarion Alexandru <<a href="mailto:viorel.visarion@gmail.com">viorel.visarion@gmail.com</a>><br>
><br>
> Learn to get/set the listen devices that vnc is using.<br>
><br>
> When setting the listen devices, first remove the 'listen' attribute<br>
> to avoid inconsistencies checks between the 'listen' attribute and the<br>
> 'address' attribute of the 'listen' node.<br>
> ---<br>
>  .../libvirt-gconfig-domain-<wbr>graphics-vnc.c          | 84 ++++++++++++++++++++++<br>
>  .../libvirt-gconfig-domain-<wbr>graphics-vnc.h          |  4 ++<br>
>  libvirt-gconfig/libvirt-<wbr>gconfig.sym                |  2 +<br>
>  3 files changed, 90 insertions(+)<br>
><br>
> diff --git a/libvirt-gconfig/libvirt-<wbr>gconfig-domain-graphics-vnc.c b/libvirt-gconfig/libvirt-<wbr>gconfig-domain-graphics-vnc.c<br>
> index fc26bb9..46f3a4b 100644<br>
> --- a/libvirt-gconfig/libvirt-<wbr>gconfig-domain-graphics-vnc.c<br>
> +++ b/libvirt-gconfig/libvirt-<wbr>gconfig-domain-graphics-vnc.c<br>
> @@ -120,6 +120,90 @@ void gvir_config_domain_graphics_<wbr>vnc_set_port(<wbr>GVirConfigDomainGraphicsVnc *graph<br>
>                                                 NULL);<br>
>  }<br>
><br>
> +/**<br>
> + * gvir_config_domain_graphics_<wbr>vnc_set_listen_devices:<br>
> + * @graphics: a #GVirConfigDomainGraphicsVnc<br>
> + * @listens: (in) (element-type LibvirtGConfig.<wbr>DomainGraphicsListen):<br>
> + *<br>
> + * This method is used to set the various listen nodes a #GVirConfigDomainGraphicsVnc<br>
> + * device can handle.<br>
> +*/<br>
> +void gvir_config_domain_graphics_<wbr>vnc_set_listen_devices(<wbr>GVirConfigDomainGraphicsVnc *graphics,<br>
> +                                                        GList *listens)<br>
<br>
</div></div>Patch is good overall, but the naming needs improving, these <listen><br>
nodes are not devices. I'm not exactly sure how to name the API though.<br>
_set_listen_nodes() is the only thing which comes to mind, and I don't<br>
think it's great. Another options would be just _set_listens(), but<br>
that's odd too. Any better suggestion?<br>
<br>
Apart from this naming issue (which has to be fixed before pushing),<br>
<br>
Acked-by: Christophe Fergeau <<a href="mailto:cfergeau@redhat.com">cfergeau@redhat.com</a>><br>
<span class="HOEnZb"><font color="#888888"><br>
Christophe<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr">Visarion-Mingopol Alexandru-Viorel<div><div>Telefon : 0729614060</div></div><div>Best Bucuresti</div></div></div></div></div>
</div>