<div dir="ltr">Hi Christophe,<br><br><div><div class="gmail_extra"><div class="gmail_quote">On Mon, Dec 9, 2013 at 11:35 AM, 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Ah cool, thanks. This mostly looks good, though I'd probably add a check<br>
that the property has the right type using g_object_class_find_property.<br></blockquote><div><br></div><div>That's a good idea.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<div class="im"><br>
> I also switched out the two G_MAXUINT's in OvirtVmPool for G_MAXUINT32's.<br>
<br>
</div>Hmm, I'm not sure about that one, what is the reasoning for it? I'd move<br>
this to a different commit.<br></blockquote><div><br></div><div>Fair point about the different commit. I was aligning them to your choice of making ovirt_utils_guint_from_string() return false if value64 exceeds G_MAXUINT32.<br>
<br></div><div>This is a quick change, so I'll get you a new patch shortly.<br></div><div><br>Thanks!<br></div><div>iordan<br></div></div></div></div></div>