From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53614) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XXagq-0007Eu-8v for qemu-devel@nongnu.org; Fri, 26 Sep 2014 14:56:30 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XXagk-0001hz-3F for qemu-devel@nongnu.org; Fri, 26 Sep 2014 14:56:24 -0400 Received: from mx1.redhat.com ([209.132.183.28]:4040) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XXagj-0001hd-RZ for qemu-devel@nongnu.org; Fri, 26 Sep 2014 14:56:18 -0400 Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s8QIuCu5010580 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Fri, 26 Sep 2014 14:56:12 -0400 Date: Fri, 26 Sep 2014 13:37:21 -0400 From: Luiz Capitulino Message-ID: <20140926133721.5f5690d2@redhat.com> In-Reply-To: <1411627771-13168-1-git-send-email-armbru@redhat.com> References: <1411627771-13168-1-git-send-email-armbru@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] qemu-socket: Eliminate silly QERR_ macros List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Markus Armbruster Cc: qemu-devel@nongnu.org On Thu, 25 Sep 2014 08:49:31 +0200 Markus Armbruster wrote: > The QERR_ macros are leftovers from the days of "rich" error objects. > They're used with error_set() and qerror_report(), and expand into the > first *two* arguments. This trickiness has become pointless. Clean > up. > > Signed-off-by: Markus Armbruster Applied to the qmp branch, thanks. > --- > include/qapi/qmp/qerror.h | 12 ------------ > util/qemu-sockets.c | 26 +++++++++++++------------- > 2 files changed, 13 insertions(+), 25 deletions(-) > > diff --git a/include/qapi/qmp/qerror.h b/include/qapi/qmp/qerror.h > index 774e75d..0ca6cbd 100644 > --- a/include/qapi/qmp/qerror.h > +++ b/include/qapi/qmp/qerror.h > @@ -154,16 +154,4 @@ void qerror_report_err(Error *err); > #define QERR_UNSUPPORTED \ > ERROR_CLASS_GENERIC_ERROR, "this feature or command is not currently supported" > > -#define QERR_SOCKET_CONNECT_FAILED \ > - ERROR_CLASS_GENERIC_ERROR, "Failed to connect socket" > - > -#define QERR_SOCKET_LISTEN_FAILED \ > - ERROR_CLASS_GENERIC_ERROR, "Failed to listen on socket" > - > -#define QERR_SOCKET_BIND_FAILED \ > - ERROR_CLASS_GENERIC_ERROR, "Failed to bind socket" > - > -#define QERR_SOCKET_CREATE_FAILED \ > - ERROR_CLASS_GENERIC_ERROR, "Failed to create socket" > - > #endif /* QERROR_H */ > diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c > index 4a25585..1eef590 100644 > --- a/util/qemu-sockets.c > +++ b/util/qemu-sockets.c > @@ -159,7 +159,7 @@ int inet_listen_opts(QemuOpts *opts, int port_offset, Error **errp) > slisten = qemu_socket(e->ai_family, e->ai_socktype, e->ai_protocol); > if (slisten < 0) { > if (!e->ai_next) { > - error_set_errno(errp, errno, QERR_SOCKET_CREATE_FAILED); > + error_setg_errno(errp, errno, "Failed to create socket"); > } > continue; > } > @@ -183,7 +183,7 @@ int inet_listen_opts(QemuOpts *opts, int port_offset, Error **errp) > } > if (p == port_max) { > if (!e->ai_next) { > - error_set_errno(errp, errno, QERR_SOCKET_BIND_FAILED); > + error_setg_errno(errp, errno, "Failed to bind socket"); > } > } > } > @@ -194,7 +194,7 @@ int inet_listen_opts(QemuOpts *opts, int port_offset, Error **errp) > > listen: > if (listen(slisten,1) != 0) { > - error_set_errno(errp, errno, QERR_SOCKET_LISTEN_FAILED); > + error_setg_errno(errp, errno, "Failed to listen on socket"); > closesocket(slisten); > freeaddrinfo(res); > return -1; > @@ -281,7 +281,7 @@ static int inet_connect_addr(struct addrinfo *addr, bool *in_progress, > > sock = qemu_socket(addr->ai_family, addr->ai_socktype, addr->ai_protocol); > if (sock < 0) { > - error_set_errno(errp, errno, QERR_SOCKET_CREATE_FAILED); > + error_setg_errno(errp, errno, "Failed to create socket"); > return -1; > } > socket_set_fast_reuse(sock); > @@ -302,7 +302,7 @@ static int inet_connect_addr(struct addrinfo *addr, bool *in_progress, > connect_state); > *in_progress = true; > } else if (rc < 0) { > - error_set_errno(errp, errno, QERR_SOCKET_CONNECT_FAILED); > + error_setg_errno(errp, errno, "Failed to connect socket"); > closesocket(sock); > return -1; > } > @@ -466,20 +466,20 @@ int inet_dgram_opts(QemuOpts *opts, Error **errp) > /* create socket */ > sock = qemu_socket(peer->ai_family, peer->ai_socktype, peer->ai_protocol); > if (sock < 0) { > - error_set_errno(errp, errno, QERR_SOCKET_CREATE_FAILED); > + error_setg_errno(errp, errno, "Failed to create socket"); > goto err; > } > socket_set_fast_reuse(sock); > > /* bind socket */ > if (bind(sock, local->ai_addr, local->ai_addrlen) < 0) { > - error_set_errno(errp, errno, QERR_SOCKET_BIND_FAILED); > + error_setg_errno(errp, errno, "Failed to bind socket"); > goto err; > } > > /* connect to peer */ > if (connect(sock,peer->ai_addr,peer->ai_addrlen) < 0) { > - error_set_errno(errp, errno, QERR_SOCKET_CONNECT_FAILED); > + error_setg_errno(errp, errno, "Failed to connect socket"); > goto err; > } > > @@ -684,7 +684,7 @@ int unix_listen_opts(QemuOpts *opts, Error **errp) > > sock = qemu_socket(PF_UNIX, SOCK_STREAM, 0); > if (sock < 0) { > - error_set_errno(errp, errno, QERR_SOCKET_CREATE_FAILED); > + error_setg_errno(errp, errno, "Failed to create socket"); > return -1; > } > > @@ -709,11 +709,11 @@ int unix_listen_opts(QemuOpts *opts, Error **errp) > > unlink(un.sun_path); > if (bind(sock, (struct sockaddr*) &un, sizeof(un)) < 0) { > - error_set_errno(errp, errno, QERR_SOCKET_BIND_FAILED); > + error_setg_errno(errp, errno, "Failed to bind socket"); > goto err; > } > if (listen(sock, 1) < 0) { > - error_set_errno(errp, errno, QERR_SOCKET_LISTEN_FAILED); > + error_setg_errno(errp, errno, "Failed to listen on socket"); > goto err; > } > > @@ -739,7 +739,7 @@ int unix_connect_opts(QemuOpts *opts, Error **errp, > > sock = qemu_socket(PF_UNIX, SOCK_STREAM, 0); > if (sock < 0) { > - error_set_errno(errp, errno, QERR_SOCKET_CREATE_FAILED); > + error_setg_errno(errp, errno, "Failed to create socket"); > return -1; > } > if (callback != NULL) { > @@ -774,7 +774,7 @@ int unix_connect_opts(QemuOpts *opts, Error **errp, > } > > if (rc < 0) { > - error_set_errno(errp, -rc, QERR_SOCKET_CONNECT_FAILED); > + error_setg_errno(errp, -rc, "Failed to connect socket"); > close(sock); > sock = -1; > }