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 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).