qemu-devel
[Top][All Lists]
Advanced

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

[Qemu-devel] [PATCH 0/9]: qapi: generate qerrors from qapi-schema-errors


From: Luiz Capitulino
Subject: [Qemu-devel] [PATCH 0/9]: qapi: generate qerrors from qapi-schema-errors.json
Date: Wed, 18 Jul 2012 14:49:11 -0300

This series moves all qerrors we have today to qapi-schema-errors.json and
generate the error macros and table from it.

With this series, one doesn't have to manually add an error macro and the
matching table entry anymore. He or she just have to add the new error to
qapi-schema-errors.json.

There's only one small problem: the matching between error class name and
the (generated) error macro may not be clear for those not familirized with
qerrors. There are two possible solutions to this:

  1. Add the generated macro name along with the error class name in
     qapi-schema-json-errors.json; and/or

  2. add docs/qapi-errors.txt to explain this in detail

This series is my first step on improving our error API.

 Makefile                 |   8 +-
 hw/qdev-properties.c     |   2 +-
 migration-tcp.c          |   6 +-
 monitor.c                |   2 +-
 qapi-schema-errors.json  | 616 +++++++++++++++++++++++++++++++++++++++++++++++
 qapi/qmp-dispatch.c      |   2 +-
 qapi/qmp-input-visitor.c |   2 +-
 qemu-sockets.c           |  22 +-
 qerror.c                 | 310 +-----------------------
 qerror.h                 | 220 +----------------
 scripts/check-qerror.sh  |   6 +-
 scripts/qapi-errors.py   | 180 ++++++++++++++
 scripts/qapi.py          |   4 +-
 13 files changed, 827 insertions(+), 553 deletions(-)



reply via email to

[Prev in Thread] Current Thread [Next in Thread]