[libvirt] [PATCH 16/21] admin: Generate ConnectListServers dispatch helpers
Ján Tomko
jtomko at redhat.com
Thu Mar 10 13:57:39 UTC 2016
On Thu, Mar 10, 2016 at 05:54:05AM +0100, Martin Kletzander wrote:
> Since we have the opportunity now, let's save some precious code lines.
>
> Signed-off-by: Martin Kletzander <mkletzan at redhat.com>
> ---
> daemon/admin.c | 45 ---------------------------------
> daemon/admin_server.c | 6 ++---
> daemon/admin_server.h | 6 ++---
> po/POTFILES.in | 1 -
> src/admin/admin_protocol.x | 4 +--
> src/admin/admin_remote.c | 63 ----------------------------------------------
> 6 files changed, 8 insertions(+), 117 deletions(-)
>
ACK, beautiful diffstat.
The new generated version of adminDispatchConnectListServers has
the following check:
+ if (!priv->dmn) {
+ virReportError(VIR_ERR_INTERNAL_ERROR, "%s", _("connection not open"));
+ goto cleanup;
+ }
Is it dead code or has it been missing in the manual implementation?
Jan
More information about the libvir-list
mailing list