From: Luiz Capitulino <lcapitulino@redhat.com>
To: Vincent Hanquez <tab@snarc.org>
Cc: aliguori@us.ibm.com, qemu-devel@nongnu.org, avi@redhat.com
Subject: Re: [Qemu-devel] [PATCH v2 00/29] QMonitor
Date: Fri, 28 Aug 2009 10:22:02 -0300 [thread overview]
Message-ID: <20090828102202.2ef59eee@doriath> (raw)
In-Reply-To: <20090828114334.GA3107@snarc.org>
On Fri, 28 Aug 2009 12:43:35 +0100
Vincent Hanquez <tab@snarc.org> wrote:
> On Wed, Aug 26, 2009 at 02:05:23PM -0300, Luiz Capitulino wrote:
> > Hi there,
> >
> > This new version of the QMonitor series has more simplifications,
> > renames and improvements.
> >
> > Most changes are because of simplifications on QInt, please review
> > carefully its implementation and usage.
> >
> > Changelog, description and diffstat follows.
>
> Hi Luiz,
>
> This is a quite nice serie of patches ! just a single question with the
> direction of this refactoring...
>
> Are you going to also wrap the monitor functions return values into the Q* types
> as well ? that would make the monitor transport (whatever it is) completely
> separated from the monitor functions and quite nice from a modularity PoV.
Yes, that's the goal.
All data that monitor functions print today will be returned to the
monitor as QObjects, then the monitor will decide to print them
in a user format or emit them as a machine protocol.
prev parent reply other threads:[~2009-08-28 13:22 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-26 17:05 [Qemu-devel] [PATCH v2 00/29] QMonitor Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 01/29] Introduce QObject Luiz Capitulino
2009-08-26 18:12 ` [Qemu-devel] " Paolo Bonzini
2009-08-26 17:05 ` [Qemu-devel] [PATCH 02/29] Introduce QInt Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 03/29] Introduce QString Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 04/29] Introduce QDict Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 05/29] Add wrappers to functions used by the Monitor Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 06/29] monitor: New format for handlers argument types Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 07/29] monitor: Setup a QDict with arguments to handlers Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 08/29] monitor: Export QDict header Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 09/29] monitor: Port handler_0 to use QDict Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 10/29] monitor: Port handler_1 " Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 11/29] monitor: Port handler_2 " Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 12/29] monitor: Port handler_3 " Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 13/29] monitor: Port handler_4 " Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 14/29] monitor: Port handler_5 " Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 15/29] monitor: Port handler_6 " Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 16/29] monitor: Port handler_7 " Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 17/29] monitor: Drop handler_8 and handler_9 Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 18/29] monitor: Port handler_10 to use QDict Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 19/29] monitor: Split monitor_handle_command() Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 20/29] monitor: Drop unused macros Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 21/29] monitor: Drop str_allocated[] Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 22/29] monitor: Drop args[] handling code Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 23/29] monitor: fail when 'i' type is greater than 32-bit Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 24/29] monitor: Update supported types documentation Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 25/29] Add check support Luiz Capitulino
2009-08-26 19:34 ` [Qemu-devel] " Juan Quintela
2009-08-26 20:39 ` Luiz Capitulino
2009-08-26 21:53 ` Juan Quintela
2009-08-26 17:05 ` [Qemu-devel] [PATCH 26/29] Introduce QInt unit-tests Luiz Capitulino
[not found] ` <m3ocq2dyu2.fsf@neno.mitica>
2009-08-28 18:26 ` [Qemu-devel] " Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 27/29] Introduce QString unit-tests Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 28/29] Introduce QDict test data file Luiz Capitulino
2009-08-26 17:05 ` [Qemu-devel] [PATCH 29/29] Introduce QDict unit-tests Luiz Capitulino
2009-08-28 11:43 ` [Qemu-devel] [PATCH v2 00/29] QMonitor Vincent Hanquez
2009-08-28 13:22 ` Luiz Capitulino [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=20090828102202.2ef59eee@doriath \
--to=lcapitulino@redhat.com \
--cc=aliguori@us.ibm.com \
--cc=avi@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=tab@snarc.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.