[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/4] scripts: qmp-shell: Expand support for Q
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/4] scripts: qmp-shell: Expand support for QMP expressions |
Date: |
Wed, 22 Apr 2015 11:25:02 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 |
On 04/22/2015 11:18 AM, Eric Blake wrote:
> On 04/22/2015 10:21 AM, John Snow wrote:
>> This includes support for [] expressions, single-quotes in
>> QMP expressions (which is not strictly a part of JSON), and
>> the ability to use "True" or "False" literals instead of
>> JSON's lowercased 'true' and 'false' literals.
>>
>>
>> Signed-off-by: John Snow <address@hidden>
>> ---
>> scripts/qmp/qmp-shell | 44 ++++++++++++++++++++++++++++----------------
>> 1 file changed, 28 insertions(+), 16 deletions(-)
>
> As far as I can tell, this makes qmp-shell strictly more useful, so I
> have no qualms giving:
>
> Reviewed-by: Eric Blake <address@hidden>
When combined with your one-line touchup, of course.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v2 0/4] scripts: qmp-shell: add transaction support, John Snow, 2015/04/22
- [Qemu-devel] [PATCH v2 2/4] scripts: qmp-shell: Expand support for QMP expressions, John Snow, 2015/04/22
- [Qemu-devel] [PATCH v2 4/4] scripts: qmp-shell: Add verbose flag, John Snow, 2015/04/22
- [Qemu-devel] [PATCH v2 3/4] scripts: qmp-shell: add transaction subshell, John Snow, 2015/04/22
- [Qemu-devel] [PATCH v2 1/4] scripts: qmp-shell: refactor helpers, John Snow, 2015/04/22
- Re: [Qemu-devel] [PATCH v2 0/4] scripts: qmp-shell: add transaction support, John Snow, 2015/04/22