<div dir="ltr">ping</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Sep 8, 2013 at 2:42 PM, Osier Yang <span dir="ltr"><<a href="mailto:jyang@redhat.com" target="_blank">jyang@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="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+ *<br>
+ * If 0 is passed as @flags, libvirt will choose the best way, and won't<br>
+ * include agent in it.<br>
</blockquote>
<br>
I'm thinking if we don't need to be so complicated, by defaulting to one<br>
of _LEASE or _SNOOP explicitly.<br>
<br>
That says (conclusion of all of above), I'm wondering if define the enum like<br>
blow is better.<br>
<br>
<br>
typedef enum {<br>
    VIR_DOMAIN_INTERFACE_<u></u>ADDRESSES_AGENT = (1 << 1), /* Query qemu guest agent */<br>
</blockquote>
<br>
</blockquote>
<br></div>
And s/AGENT/GUEST_AGENT/, since "AGENT" could mean too much than "guest agent"<br>
<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Nehal J Wani<br>
UG3, BTech CS+MS(CL)<br>
IIIT-Hyderabad<div><a href="http://commandlinewani.blogspot.com" target="_blank">http://commandlinewani.blogspot.com</a></div></div>
</div>