From: Hani Benhabiles <kroosec@gmail.com>
To: Luiz Capitulino <lcapitulino@redhat.com>
Cc: imammedo@redhat.com, qemu-devel@nongnu.org, pbonzini@redhat.com
Subject: Re: [Qemu-devel] [PATCH 0/4] monitor: Add argument completion for multiple commands.
Date: Tue, 11 Feb 2014 23:10:10 +0100 [thread overview]
Message-ID: <20140211221010.GC13922@Inspiron-3521> (raw)
In-Reply-To: <20140211133913.56ebf266@redhat.com>
On Tue, Feb 11, 2014 at 01:39:13PM -0500, Luiz Capitulino wrote:
> On Thu, 6 Feb 2014 23:30:09 +0100
> Hani Benhabiles <kroosec@gmail.com> wrote:
>
> > This patch series add argument completion to multiple human monitor console
> > commands.
> >
> > For device_add and object_add, only the 1st argument is completed.
> >
> > Hani Benhabiles (4):
> > monitor: Add device_del id argument completion.
> > monitor: Add device_add device argument completion.
> > monitor: Add object_del id argument completion.
> > monitor: Add object_add class argument completion.
> >
> > monitor.c | 95 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > 1 file changed, 95 insertions(+)
>
> Series look good and I've applied it to my queue, but I need your sob
> before sending a pull request.
Thanks Luiz! Sorry about the s-o-b.
Signed-off-by: Hani Benhabiles <hani@linux.com>
prev parent reply other threads:[~2014-02-11 22:10 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-06 22:30 [Qemu-devel] [PATCH 0/4] monitor: Add argument completion for multiple commands Hani Benhabiles
2014-02-06 22:30 ` [Qemu-devel] [PATCH 1/4] monitor: Add device_del id argument completion Hani Benhabiles
2014-02-11 18:35 ` Luiz Capitulino
2014-02-06 22:30 ` [Qemu-devel] [PATCH 2/4] monitor: Add device_add device " Hani Benhabiles
2014-02-06 22:30 ` [Qemu-devel] [PATCH 3/4] monitor: Add object_del id " Hani Benhabiles
2014-02-06 22:30 ` [Qemu-devel] [PATCH 4/4] monitor: Add object_add class " Hani Benhabiles
2014-02-11 18:38 ` Luiz Capitulino
2014-02-11 22:14 ` Hani Benhabiles
2014-02-11 18:39 ` [Qemu-devel] [PATCH 0/4] monitor: Add argument completion for multiple commands Luiz Capitulino
2014-02-11 22:10 ` Hani Benhabiles [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140211221010.GC13922@Inspiron-3521 \
--to=kroosec@gmail.com \
--cc=imammedo@redhat.com \
--cc=lcapitulino@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.