[Qemu-devel] Re: [PATCH 28/50] error: Let converted handlers print in human monitor

[Luiz Capitulino]

On Thu,  4 Mar 2010 16:56:49 +0100
Markus Armbruster <address@hidden> wrote:

> While fully converted handlers are not supposed to print anything when
> running in a QMP monitor, they are free to print in a human monitor.

 I disagree.

 One of the key decisions behind the new Monitor design is that handlers
are part of common code. User printing is output-specific and should not
be done by handlers.

 One of the problems with this layer violation is that the consumer of this
data might not be what you have assumed. For example, it has been suggested
that we could move the qemu shell out of qemu in the future. In this case,
this kind of information _could_ be useful.

 Another important problem is that free printing like this is error-prone
as it's not at all clear what could be printed and there's no way to catch
mistakes mechanically. Not to mention that this is for sure going to used
by those who want to ignore QMP completely.

 I'm ok with shallow conversion, provided that handler_audit() is there to
warn us about the job to be done.