Revert "admin: Add virAdmHello function"

This reverts commit 5792fabb7b.

I mistakenly pushed it along with the Admin API series.

daemon/admin_server.c

@@ -114,27 +114,4 @@ adminDispatchConnectClose(virNetServerPtr server ATTRIBUTE_UNUSED,
     return 0;
 }
 
-
-static char *
-admHello(virNetDaemonPtr dmn,
-         unsigned int flags)
-{
-    char *ret = NULL;
-    virNetServerPtr srv = NULL;
-    size_t nclients;
-
-    virCheckFlags(0, NULL);
-
-    if (!(srv = virNetDaemonGetServer(dmn, 0)))
-        return NULL;
-
-    nclients = virNetServerGetNClients(srv);
-    if (virAsprintf(&ret, "Clients: %zu", nclients) < 0)
-        goto cleanup;
-
- cleanup:
-    virObjectUnref(srv);
-    return ret;
-}
-
 #include "admin_dispatch.h"

include/libvirt/libvirt-admin.h

@@ -54,7 +54,6 @@ virAdmConnectPtr virAdmConnectOpen(const char *name, unsigned int flags);
 int virAdmConnectClose(virAdmConnectPtr conn);
 
 int virAdmConnectRef(virAdmConnectPtr conn);
-char *virAdmHello(virAdmConnectPtr conn, unsigned int flags);
 
 # ifdef __cplusplus
 }

po/POTFILES.in

@@ -1,4 +1,3 @@
-daemon/admin_dispatch.h
 daemon/admin_server.c
 daemon/libvirtd-config.c
 daemon/libvirtd.c

src/admin/admin_protocol.x

@@ -43,15 +43,6 @@ struct admin_connect_open_args {
     unsigned int flags;
 };
 
-struct admin_hello_args {
-    unsigned int flags;
-};
-
-struct admin_hello_ret {
-    admin_string greeting;
-};
-
-
 /* Define the program number, protocol version and procedure numbers here. */
 const ADMIN_PROGRAM = 0x06900690;
 const ADMIN_PROTOCOL_VERSION = 1;
@@ -82,10 +73,5 @@ enum admin_procedure {
     /**
      * @generate: client
      */
-    ADMIN_PROC_CONNECT_CLOSE = 2,
-
-    /**
-     * @generate: both
-     */
-    ADMIN_PROC_HELLO = 3
+    ADMIN_PROC_CONNECT_CLOSE = 2
 };

src/admin_protocol-structs

@@ -2,14 +2,7 @@
 struct admin_connect_open_args {
         u_int                      flags;
 };
-struct admin_hello_args {
-        u_int                      flags;
-};
-struct admin_hello_ret {
-        admin_string               greeting;
-};
 enum admin_procedure {
         ADMIN_PROC_CONNECT_OPEN = 1,
         ADMIN_PROC_CONNECT_CLOSE = 2,
-        ADMIN_PROC_HELLO = 3,
 };

src/libvirt-admin.c

@@ -384,29 +384,3 @@ virAdmConnectRef(virAdmConnectPtr conn)
 
     return 0;
 }
-
-
-/**
- * virAdmHello:
- * @conn: a connection object
- * @flags: unused, 0 for now
- *
- * Testing function.
- *
- * Returns the number of connected clients as a string.  Yes, as a
- * string.  Because it's just for testing.
- */
-char *
-virAdmHello(virAdmConnectPtr conn,
-            unsigned int flags)
-{
-    char *ret = NULL;
-
-    virResetLastError();
-
-    ret = remoteAdminHello(conn, flags);
-    if (!ret)
-        virDispatchError(NULL);
-
-    return ret;
-}

src/libvirt_admin.syms

@@ -15,5 +15,4 @@ LIBVIRT_ADMIN_1.3.0 {
         virAdmConnectOpen;
         virAdmConnectClose;
         virAdmConnectRef;
-        virAdmHello;
 };