From patchwork Wed Jun 5 12:19:26 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kevin Wolf X-Patchwork-Id: 249007 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 247D32C009E for ; Wed, 5 Jun 2013 22:20:34 +1000 (EST) Received: from localhost ([::1]:49765 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UkChc-0004CZ-3T for incoming@patchwork.ozlabs.org; Wed, 05 Jun 2013 08:20:32 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58019) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UkCh0-00048D-Gt for qemu-devel@nongnu.org; Wed, 05 Jun 2013 08:20:00 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UkCgv-0004LZ-1q for qemu-devel@nongnu.org; Wed, 05 Jun 2013 08:19:54 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49092) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UkCgu-0004LP-OQ for qemu-devel@nongnu.org; Wed, 05 Jun 2013 08:19:48 -0400 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id r55CJlmX023785 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Wed, 5 Jun 2013 08:19:48 -0400 Received: from dhcp-200-207.str.redhat.com (dhcp-192-246.str.redhat.com [10.33.192.246]) by int-mx09.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id r55CJifv018138; Wed, 5 Jun 2013 08:19:46 -0400 From: Kevin Wolf To: qemu-devel@nongnu.org Date: Wed, 5 Jun 2013 14:19:26 +0200 Message-Id: <1370434781-28570-2-git-send-email-kwolf@redhat.com> In-Reply-To: <1370434781-28570-1-git-send-email-kwolf@redhat.com> References: <1370434781-28570-1-git-send-email-kwolf@redhat.com> X-Scanned-By: MIMEDefang 2.68 on 10.5.11.22 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x X-Received-From: 209.132.183.28 Cc: kwolf@redhat.com, pbonzini@redhat.com, stefanha@redhat.com, lcapitulino@redhat.com Subject: [Qemu-devel] [PATCH v2 01/16] qemu-io: Remove unused args_command X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org The original intention seems to be something with handling multiple images at once, but this has never been implemented and the only function ever registered is implemented to make everything behave like a "global" command. Just do that unconditionally now. Signed-off-by: Kevin Wolf Reviewed-by: Eric Blake --- cmd.c | 28 ++-------------------------- cmd.h | 2 -- qemu-io.c | 10 ---------- 3 files changed, 2 insertions(+), 38 deletions(-) diff --git a/cmd.c b/cmd.c index 10a8688..4e7579b 100644 --- a/cmd.c +++ b/cmd.c @@ -34,7 +34,6 @@ cmdinfo_t *cmdtab; int ncmds; -static argsfunc_t args_func; static checkfunc_t check_func; static int ncmdline; static char **cmdline; @@ -127,22 +126,6 @@ void add_user_command(char *optarg) cmdline[ncmdline-1] = optarg; } -static int -args_command( - int index) -{ - if (args_func) - return args_func(index); - return 0; -} - -void -add_args_command( - argsfunc_t af) -{ - args_func = af; -} - static void prep_fetchline(void *opaque) { int *fetchable = opaque; @@ -155,7 +138,7 @@ static char *get_prompt(void); void command_loop(void) { - int c, i, j = 0, done = 0, fetchable = 0, prompted = 0; + int c, i, done = 0, fetchable = 0, prompted = 0; char *input; char **v; const cmdinfo_t *ct; @@ -171,14 +154,7 @@ void command_loop(void) if (c) { ct = find_command(v[0]); if (ct) { - if (ct->flags & CMD_FLAG_GLOBAL) { - done = command(ct, c, v); - } else { - j = 0; - while (!done && (j = args_command(j))) { - done = command(ct, c, v); - } - } + done = command(ct, c, v); } else { fprintf(stderr, _("command \"%s\" not found\n"), v[0]); } diff --git a/cmd.h b/cmd.h index b763b19..8e6f753 100644 --- a/cmd.h +++ b/cmd.h @@ -41,12 +41,10 @@ extern int ncmds; void help_init(void); void quit_init(void); -typedef int (*argsfunc_t)(int index); typedef int (*checkfunc_t)(const cmdinfo_t *ci); void add_command(const cmdinfo_t *ci); void add_user_command(char *optarg); -void add_args_command(argsfunc_t af); void add_check_command(checkfunc_t cf); const cmdinfo_t *find_command(const char *cmd); diff --git a/qemu-io.c b/qemu-io.c index 5e6680b..4288b8c 100644 --- a/qemu-io.c +++ b/qemu-io.c @@ -1888,15 +1888,6 @@ static int open_f(int argc, char **argv) return openfile(argv[optind], flags, growable); } -static int init_args_command(int index) -{ - /* only one device allowed so far */ - if (index >= 1) { - return 0; - } - return ++index; -} - static int init_check_command(const cmdinfo_t *ct) { if (ct->flags & CMD_FLAG_GLOBAL) { @@ -2043,7 +2034,6 @@ int main(int argc, char **argv) add_command(&wait_break_cmd); add_command(&abort_cmd); - add_args_command(init_args_command); add_check_command(init_check_command); /* open the device */