<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 5, 2021, 19:49 Richard W.M. Jones <<a href="mailto:rjones@redhat.com">rjones@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
I would just say about the examples that often it's better if they are<br>
completely stand-alone (since we want people to copy and adapt the<br>
code).  That would imply not using the common/ header file, but either<br>
inlining the function or using a less efficient but simpler loop.<br>
This is also the reason why the examples don't #include <config.h><br>
<br>
Anyway the patch is fine, but IMO could be improved as above.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Good point, Inlining is_ zero sound better.</div><div dir="auto"><br></div><div dir="auto">I will post a followup patch.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
ACK.<br>
<br>
Rich.<br>
<br>
-- <br>
Richard Jones, Virtualization Group, Red Hat <a href="http://people.redhat.com/~rjones" rel="noreferrer noreferrer" target="_blank">http://people.redhat.com/~rjones</a><br>
Read my programming and virtualization blog: <a href="http://rwmj.wordpress.com" rel="noreferrer noreferrer" target="_blank">http://rwmj.wordpress.com</a><br>
virt-p2v converts physical machines to virtual machines.  Boot with a<br>
live CD or over the network (PXE) and turn machines into KVM guests.<br>
<a href="http://libguestfs.org/virt-v2v" rel="noreferrer noreferrer" target="_blank">http://libguestfs.org/virt-v2v</a><br>
<br>
</blockquote></div></div></div>