Re: [libvirt] [PATCH 1/3] Turn libvirt.c error reporting functions into macros

The virLibConnError() function (and related ones) do not correctly report line number info. Turn them all into macros so line numbers are reported correctly. Drop the connection object in all of them since it is no longer used. Also from the virLibConnWarning() equivalents completely. Now that the Xen driver is running 100% inside libvirtd, those codepaths for secondary drivers cannot be reached. * src/libvirt.c: Replace error functions with macros --- src/libvirt.c | 1708 ++++++++++++++++++++++++--------------------------------- 1 files changed, 717 insertions(+), 991 deletions(-)

@@ -1302,7 +1059,7 @@ do_open (const char *name, if (!ret->driver) { /* If we reach here, then all drivers declined the connection. */ - virLibConnError (NULL, VIR_ERR_NO_CONNECT, name); + virLibConnError(VIR_ERR_NO_CONNECT, "No connection for URI %s", NULLSTR(name));