<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Wed, Aug 8, 2018 at 6:17 PM 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">On Wed, Aug 08, 2018 at 06:08:45PM +0300, Nir Soffer wrote:<br>
> Can we simplify this these 2 calls?<br>
> <br>
>     nbdkit_error ("%s: %s: error", script, callback);<br>
>     PyErr_PrintEx (0);<br>
<br>
No, we want to ensure errors go through the nbdkit_error function so<br>
that we can in future implement syslog logging.<br>
<br>
> In this case we don't to fallback to the simple error print.<br>
<br>
Do you mean in the case of using PyErr_PrintEx or more generally<br>
falling back is a problem?<br></blockquote><div><br></div><div>Just for PyErr_PrintEx, since it cannot fail.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Rich.<br>
<br>
-- <br>
Richard Jones, Virtualization Group, Red Hat <a href="http://people.redhat.com/~rjones" rel="noreferrer" target="_blank">http://people.redhat.com/~rjones</a><br>
Read my programming and virtualization blog: <a href="http://rwmj.wordpress.com" rel="noreferrer" target="_blank">http://rwmj.wordpress.com</a><br>
virt-builder quickly builds VMs from scratch<br>
<a href="http://libguestfs.org/virt-builder.1.html" rel="noreferrer" target="_blank">http://libguestfs.org/virt-builder.1.html</a><br>
</blockquote></div></div>