<div dir="ltr"><div><div><div>Great !<br></div>Thanks for the merge and your work :)<br><br></div>Cheers,<br></div>Xavier<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 17, 2017 at 9:12 AM, Martin Kletzander <span dir="ltr"><<a href="mailto:mkletzan@redhat.com" target="_blank">mkletzan@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, May 05, 2017 at 04:39:34PM +0200, Xavier Fernandez wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
In Python 3, if the file is open in binary mode, ``got`` will end up<br>
being equal to b"" and b"" != "" in Python 3.<br>
---<br>
libvirt-override-virStream.py | 2 +-<br>
1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
</blockquote>
<br></span>
Makes sense, I also stumbled upon something similar.  ACK and I will<br>
push this in a while, thanks for the patch.  I'll reword the commit<br>
message a tiny bit.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
diff --git a/libvirt-override-virStream.p<wbr>y b/libvirt-override-virStream.p<wbr>y<br>
index c328154..2e77cc7 100644<br>
--- a/libvirt-override-virStream.p<wbr>y<br>
+++ b/libvirt-override-virStream.p<wbr>y<br>
@@ -83,7 +83,7 @@<br>
                    pass<br>
                raise e<br>
<br>
-            if got == "":<br>
+            if not got:<br>
                break<br>
<br>
            ret = self.send(got)<br>
--<br>
2.12.2<br>
<br></span><span class="HOEnZb"><font color="#888888">
--<br>
libvir-list mailing list<br>
<a href="mailto:libvir-list@redhat.com" target="_blank">libvir-list@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/libvir-list" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/libvir-list</a><br>
</font></span></blockquote>
</blockquote></div><br></div>