<div dir="ltr"><div><div>Hi Christophe,<br><br>I have another patch for you to consider. This one removes some duplicated code by creating and using a utility function to ovirt-utils.c. I also switched out the two G_MAXUINT's in OvirtVmPool for G_MAXUINT32's.<br>
<br></div>We can make use of it any time an integer attribute needs to be parsed. Maybe it should be in the OvirtResource class, but I'll leave that for you to decide. Also, feel free to change naming or anything else you don't like.<br>
<br></div>Thanks!<br>iordan<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Dec 5, 2013 at 4:05 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Wed, Dec 04, 2013 at 11:12:09PM -0500, i iordanov wrote:<br>
> On Wed, Dec 4, 2013 at 10:30 AM, Christophe Fergeau <<a href="mailto:cfergeau@redhat.com">cfergeau@redhat.com</a>>wrote:<br>
> > It would be nice if you could check I did not break anything for you (I<br>
> > haven't tested ovirt_vm_pool_activate_vm() at all). Then I can push all of<br>
> > it to git master ;)<br>
> ><br>
><br>
> I tested and it allocates VMs just as well as the before your changes. It<br>
> also reports the parsed attributes correctly for me.<br>
<br>
</div>Cool, thanks for the testing, I can push it upstream then!<br>
<span class="HOEnZb"><font color="#888888"><br>
Christophe<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>The conscious mind has only one thread of execution.
</div>