Re: [Qemu-devel] [PATCH 2/2 v2] Add the blockdev-reopen and blockdev-migrate commands

[Luiz Capitulino]

On Fri, 24 Feb 2012 21:32:30 +0100
Paolo Bonzini <address@hidden> wrote:

> On 02/24/2012 08:01 PM, Luiz Capitulino wrote:
> > IMO, you could require the filename arguments to be escaped already.
> 
> The monitor command was introduced exactly to avoid having to worry
> about details such as escaping.  JSON is already supposed to take care
> of those.

Then the escaping should be done by the parser, not by the command
implementation.

> 
> That said, I think Eric is right that we need to specify the formats, so
> we'll need another iteration, and we probably need to rename the command
> to drive-migrate too.
> 
> Paolo
>