[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] tests: Make iotest 223 easier to edit
From: |
Max Reitz |
Subject: |
Re: [PATCH 1/2] tests: Make iotest 223 easier to edit |
Date: |
Tue, 8 Oct 2019 11:04:55 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.0 |
On 07.10.19 22:06, Eric Blake wrote:
> On 10/7/19 7:05 AM, Max Reitz wrote:
>> On 24.09.19 16:35, Eric Blake wrote:
>>> Log the QMP input to qemu, not just the QMP output.
>>
>> Why not just add this functionality to _send_qemu_cmd directly? (Like
>> silent already does for replies, although it’s inverted.)
>
> Interesting idea. I'll give it a shot (it may have a larger effect on
> more .out files, but that's probably a good thing).
I was thinking about making it conditional, like it’s done with the
$silent parameter. But I mean, there is actually no good reason for it
to print the output but omit the input, I suppose... (Other than that
this change will affect many .out files, as you say)
Max
>> (Although I’m not quite sold on the indentation for commands, because
>> (1) we don’t do that in other tests, (2) I’d prefer some prefix like
>> ->/<-, and (3) there is generally no need because commands start with
>> “execute” and replies start with “return”, “error”, or “event”, so they
>> already have clear prefixes to distinguish the two classes.)
>
> I don't mind avoiding indentation; as you say, direction can be inferred
> by contents.
>
signature.asc
Description: OpenPGP digital signature