Re: [Qemu-devel] [PATCH v2 13/29] qapi: Lift error reporting from QAPISc

qemu-devel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2 13/29] qapi: Lift error reporting from QAPISc

From:	Eric Blake
Subject:	Re: [Qemu-devel] [PATCH v2 13/29] qapi: Lift error reporting from QAPISchema.__init__() to callers
Date:	Mon, 12 Feb 2018 13:52:17 -0600
User-agent:	Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0

On 02/11/2018 03:35 AM, Markus Armbruster wrote:

Signed-off-by: Markus Armbruster <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>
---
  scripts/qapi-gen.py            |  8 ++++++--
  scripts/qapi/common.py         | 23 +++++++++--------------
  tests/qapi-schema/test-qapi.py | 10 ++++++++--
  3 files changed, 23 insertions(+), 18 deletions(-)


The commit message didn't say why, but the change is reasonable.

Reviewed-by: Eric Blake <address@hidden>

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

[Prev in Thread]

Current Thread

[Next in Thread]

[Qemu-devel] [PATCH v2 08/29] qapi-gen: New common driver for code and doc generators, (continued)
- [Qemu-devel] [PATCH v2 08/29] qapi-gen: New common driver for code and doc generators, Markus Armbruster, 2018/02/11
  - Re: [Qemu-devel] [PATCH v2 08/29] qapi-gen: New common driver for code and doc generators, Eric Blake, 2018/02/12
    - Re: [Qemu-devel] [PATCH v2 08/29] qapi-gen: New common driver for code and doc generators, Markus Armbruster, 2018/02/23
    - Re: [Qemu-devel] [PATCH v2 08/29] qapi-gen: New common driver for code and doc generators, Eric Blake, 2018/02/26
  - Re: [Qemu-devel] [PATCH v2 08/29] qapi-gen: New common driver for code and doc generators, Michael Roth, 2018/02/19
- [Qemu-devel] [PATCH v2 22/29] qapi: Generate separate .h, .c for each module, Markus Armbruster, 2018/02/11
  - Re: [Qemu-devel] [PATCH v2 22/29] qapi: Generate separate .h, .c for each module, Eric Blake, 2018/02/12
    - Re: [Qemu-devel] [PATCH v2 22/29] qapi: Generate separate .h, .c for each module, Markus Armbruster, 2018/02/23
    - Re: [Qemu-devel] [PATCH v2 22/29] qapi: Generate separate .h, .c for each module, Eric Blake, 2018/02/26
- [Qemu-devel] [PATCH v2 13/29] qapi: Lift error reporting from QAPISchema.__init__() to callers, Markus Armbruster, 2018/02/11
  - Re: [Qemu-devel] [PATCH v2 13/29] qapi: Lift error reporting from QAPISchema.__init__() to callers, Eric Blake <=
  - Re: [Qemu-devel] [PATCH v2 13/29] qapi: Lift error reporting from QAPISchema.__init__() to callers, Michael Roth, 2018/02/19
- [Qemu-devel] [PATCH v2 23/29] Include less of the generated modular QAPI headers, Markus Armbruster, 2018/02/11
  - Re: [Qemu-devel] [PATCH v2 23/29] Include less of the generated modular QAPI headers, Eric Blake, 2018/02/12
    - Re: [Qemu-devel] [PATCH v2 23/29] Include less of the generated modular QAPI headers, Markus Armbruster, 2018/02/23
  - Re: [Qemu-devel] [PATCH v2 23/29] Include less of the generated modular QAPI headers, Marc-Andre Lureau, 2018/02/13
- [Qemu-devel] [PATCH v2 24/29] qapi: Empty out qapi-schema.json, Markus Armbruster, 2018/02/11
  - Re: [Qemu-devel] [PATCH v2 24/29] qapi: Empty out qapi-schema.json, Eric Blake, 2018/02/12
    - Re: [Qemu-devel] [PATCH v2 24/29] qapi: Empty out qapi-schema.json, Markus Armbruster, 2018/02/23
- Re: [Qemu-devel] [PATCH v2 00/29] Modularize generated QAPI code, no-reply, 2018/02/11
- [Qemu-devel] [PATCH v2 23.5/29] watchdog: Consolidate QAPI into single file, Eric Blake, 2018/02/26

Prev by Date: Re: [Qemu-devel] [PATCH v2 10/29] qapi: Touch generated files only when they change
Next by Date: Re: [Qemu-devel] [PATCH v2 14/29] qapi: Concentrate QAPISchemaParser.exprs updates in .__init__()
Previous by thread: [Qemu-devel] [PATCH v2 13/29] qapi: Lift error reporting from QAPISchema.__init__() to callers
Next by thread: Re: [Qemu-devel] [PATCH v2 13/29] qapi: Lift error reporting from QAPISchema.__init__() to callers
Index(es):
- Date
- Thread