qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] monitor: Remove unused bool field 'qapi' in mon_cmd_t struct
@ 2012-03-12 18:13 Jeff Cody
  2012-03-13 13:37 ` Luiz Capitulino
  2012-03-15 10:33 ` Stefan Hajnoczi
  0 siblings, 2 replies; 3+ messages in thread
From: Jeff Cody @ 2012-03-12 18:13 UTC (permalink / raw)
  To: qemu-devel; +Cc: qemu-trivial, lcapitulino

Some minor code cleanup: the 'qapi' bool field in mon_cmd_t is
unused, and can be removed.

Signed-off-by: Jeff Cody <jcody@redhat.com>
---
 monitor.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/monitor.c b/monitor.c
index a787aab..34272eb 100644
--- a/monitor.c
+++ b/monitor.c
@@ -128,7 +128,6 @@ typedef struct mon_cmd_t {
         int  (*cmd_async)(Monitor *mon, const QDict *params,
                           MonitorCompletion *cb, void *opaque);
     } mhandler;
-    bool qapi;
     int flags;
 } mon_cmd_t;
 
-- 
1.7.9.rc2.1.g69204

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [Qemu-devel] [PATCH] monitor: Remove unused bool field 'qapi' in mon_cmd_t struct
  2012-03-12 18:13 [Qemu-devel] [PATCH] monitor: Remove unused bool field 'qapi' in mon_cmd_t struct Jeff Cody
@ 2012-03-13 13:37 ` Luiz Capitulino
  2012-03-15 10:33 ` Stefan Hajnoczi
  1 sibling, 0 replies; 3+ messages in thread
From: Luiz Capitulino @ 2012-03-13 13:37 UTC (permalink / raw)
  To: Jeff Cody; +Cc: qemu-trivial, qemu-devel

On Mon, 12 Mar 2012 14:13:57 -0400
Jeff Cody <jcody@redhat.com> wrote:

> Some minor code cleanup: the 'qapi' bool field in mon_cmd_t is
> unused, and can be removed.
> 
> Signed-off-by: Jeff Cody <jcody@redhat.com>

Acked-by: Luiz Capitulino <lcapitulino@redhat.com>

> ---
>  monitor.c |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/monitor.c b/monitor.c
> index a787aab..34272eb 100644
> --- a/monitor.c
> +++ b/monitor.c
> @@ -128,7 +128,6 @@ typedef struct mon_cmd_t {
>          int  (*cmd_async)(Monitor *mon, const QDict *params,
>                            MonitorCompletion *cb, void *opaque);
>      } mhandler;
> -    bool qapi;
>      int flags;
>  } mon_cmd_t;
>  

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Qemu-devel] [PATCH] monitor: Remove unused bool field 'qapi' in mon_cmd_t struct
  2012-03-12 18:13 [Qemu-devel] [PATCH] monitor: Remove unused bool field 'qapi' in mon_cmd_t struct Jeff Cody
  2012-03-13 13:37 ` Luiz Capitulino
@ 2012-03-15 10:33 ` Stefan Hajnoczi
  1 sibling, 0 replies; 3+ messages in thread
From: Stefan Hajnoczi @ 2012-03-15 10:33 UTC (permalink / raw)
  To: Jeff Cody; +Cc: qemu-trivial, qemu-devel, lcapitulino

On Mon, Mar 12, 2012 at 02:13:57PM -0400, Jeff Cody wrote:
> Some minor code cleanup: the 'qapi' bool field in mon_cmd_t is
> unused, and can be removed.
> 
> Signed-off-by: Jeff Cody <jcody@redhat.com>
> ---
>  monitor.c |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)

Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu/commits/trivial-patches

Stefan

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-03-15 10:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-03-12 18:13 [Qemu-devel] [PATCH] monitor: Remove unused bool field 'qapi' in mon_cmd_t struct Jeff Cody
2012-03-13 13:37 ` Luiz Capitulino
2012-03-15 10:33 ` Stefan Hajnoczi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).