From: Markus Armbruster <armbru@redhat.com>
To: qemu-devel@nongnu.org
Cc: marcandre.lureau@redhat.com, mdroth@linux.vnet.ibm.com
Subject: [Qemu-devel] [PATCH v3 00/16] qapi: Schema language cleanups & doc improvements
Date: Fri, 13 Sep 2019 22:13:33 +0200 [thread overview]
Message-ID: <20190913201349.24332-1-armbru@redhat.com> (raw)
v3:
* PATCH 05
- Typo fixed [Eric]
* PATCH 06+07
- Additional comments [Eric]
* PATCH 11
- Replace one more QAPISchemaMember by QAPISchemaEnumMember
* PATCH 13+15
- Doc phrasing tweaks [Eric]
* PATCH 14+15
- Belatedly update for v2's restriction to printable ASCII [Eric]
- Correct claim "order of top-level expression doesn't matter" [Eric]
* PATCH 15
- Fix EBNF for PRAGMA [Eric]
v2:
* PATCH v1 05: Dropped
* PATCH 01,05,07-11,16: New
* PATCH 03:
- Commit message typo fixed
- Assertions tightened
* PATCH 04:
- Document member name 'u' is reserved
- Drop an accidental change to section "Enumeration types"
- Typo fixed
* PATCH 06:
- Outlaw anything but printable ASCII, not just control characters
* PATCH 15:
- Adopt more convenient terminology: directive, definition
- Explanation of grammar notation missed grouping
- Drop a more text redundant with the grammar
- Drop more text redundant with section "Naming rules and reserved
names"
- Turn "boxed needs data: STRING" into syntax
- Improve section "Documentation comments" some more
- Minor phrasing tweaks
Markus Armbruster (16):
scripts/git.orderfile: Match QAPI schema more precisely
qapi: Drop check_type()'s redundant parameter @allow_optional
qapi: Drop support for boxed alternate arguments
docs/devel/qapi-code-gen: Minor specification fixes
tests/qapi-schema: Demonstrate bad reporting of funny characters
qapi: Restrict strings to printable ASCII
qapi: Drop support for escape sequences other than \\
qapi: Permit 'boxed' with empty type
qapi: Permit alternates with just one branch
qapi: Permit omitting all flat union branches
qapi: Adjust frontend errors to say enum value, not member
docs/devel/qapi-code-gen: Reorder sections for readability
docs/devel/qapi-code-gen: Rewrite compatibility considerations
docs/devel/qapi-code-gen: Rewrite introduction to schema
docs/devel/qapi-code-gen: Improve QAPI schema language doc
qapi: Tweak code to match docs/devel/qapi-code-gen.txt
docs/devel/qapi-code-gen.txt | 1071 ++++++++++-------
tests/test-qmp-cmds.c | 4 +
scripts/git.orderfile | 4 +-
scripts/qapi/commands.py | 4 +-
scripts/qapi/common.py | 147 +--
scripts/qapi/events.py | 12 +-
tests/Makefile.include | 7 +-
tests/qapi-schema/alternate-empty.err | 2 +-
tests/qapi-schema/alternate-empty.json | 4 +-
tests/qapi-schema/args-boxed-empty.err | 1 -
tests/qapi-schema/args-boxed-empty.json | 3 -
tests/qapi-schema/args-invalid.err | 2 +-
tests/qapi-schema/doc-missing.err | 2 +-
tests/qapi-schema/doc-no-symbol.err | 2 +-
tests/qapi-schema/enum-bad-name.err | 2 +-
tests/qapi-schema/enum-bad-name.json | 3 +-
tests/qapi-schema/enum-clash-member.err | 2 +-
.../qapi-schema/enum-dict-member-unknown.err | 2 +-
tests/qapi-schema/enum-member-case.err | 2 +-
tests/qapi-schema/escape-outside-string.err | 1 -
tests/qapi-schema/escape-outside-string.json | 3 -
tests/qapi-schema/escape-too-big.err | 1 -
tests/qapi-schema/escape-too-big.exit | 1 -
tests/qapi-schema/escape-too-big.json | 3 -
tests/qapi-schema/escape-too-big.out | 0
tests/qapi-schema/escape-too-short.err | 1 -
tests/qapi-schema/escape-too-short.exit | 1 -
tests/qapi-schema/escape-too-short.json | 3 -
tests/qapi-schema/escape-too-short.out | 0
tests/qapi-schema/flat-union-empty.err | 2 +-
tests/qapi-schema/flat-union-empty.json | 2 +-
tests/qapi-schema/ident-with-escape.err | 1 +
tests/qapi-schema/ident-with-escape.exit | 2 +-
tests/qapi-schema/ident-with-escape.json | 2 +-
tests/qapi-schema/ident-with-escape.out | 16 -
tests/qapi-schema/pragma-non-dict.err | 2 +-
tests/qapi-schema/qapi-schema-test.json | 13 +-
tests/qapi-schema/qapi-schema-test.out | 21 +-
tests/qapi-schema/string-code-point-127.err | 1 +
...-empty.exit => string-code-point-127.exit} | 0
tests/qapi-schema/string-code-point-127.json | 2 +
...ed-empty.out => string-code-point-127.out} | 0
tests/qapi-schema/string-code-point-31.err | 1 +
...-string.exit => string-code-point-31.exit} | 0
tests/qapi-schema/string-code-point-31.json | 2 +
...de-string.out => string-code-point-31.out} | 0
tests/qapi-schema/struct-data-invalid.err | 2 +-
tests/qapi-schema/unicode-str.err | 1 -
tests/qapi-schema/unicode-str.exit | 1 -
tests/qapi-schema/unicode-str.json | 2 -
tests/qapi-schema/unicode-str.out | 0
tests/qapi-schema/union-empty.err | 2 +-
tests/qapi-schema/union-empty.json | 2 +-
tests/qapi-schema/unknown-escape.json | 2 +-
54 files changed, 744 insertions(+), 625 deletions(-)
delete mode 100644 tests/qapi-schema/args-boxed-empty.err
delete mode 100644 tests/qapi-schema/args-boxed-empty.json
delete mode 100644 tests/qapi-schema/escape-outside-string.err
delete mode 100644 tests/qapi-schema/escape-outside-string.json
delete mode 100644 tests/qapi-schema/escape-too-big.err
delete mode 100644 tests/qapi-schema/escape-too-big.exit
delete mode 100644 tests/qapi-schema/escape-too-big.json
delete mode 100644 tests/qapi-schema/escape-too-big.out
delete mode 100644 tests/qapi-schema/escape-too-short.err
delete mode 100644 tests/qapi-schema/escape-too-short.exit
delete mode 100644 tests/qapi-schema/escape-too-short.json
delete mode 100644 tests/qapi-schema/escape-too-short.out
create mode 100644 tests/qapi-schema/string-code-point-127.err
rename tests/qapi-schema/{args-boxed-empty.exit => string-code-point-127.exit} (100%)
create mode 100644 tests/qapi-schema/string-code-point-127.json
rename tests/qapi-schema/{args-boxed-empty.out => string-code-point-127.out} (100%)
create mode 100644 tests/qapi-schema/string-code-point-31.err
rename tests/qapi-schema/{escape-outside-string.exit => string-code-point-31.exit} (100%)
create mode 100644 tests/qapi-schema/string-code-point-31.json
rename tests/qapi-schema/{escape-outside-string.out => string-code-point-31.out} (100%)
delete mode 100644 tests/qapi-schema/unicode-str.err
delete mode 100644 tests/qapi-schema/unicode-str.exit
delete mode 100644 tests/qapi-schema/unicode-str.json
delete mode 100644 tests/qapi-schema/unicode-str.out
--
2.21.0
next reply other threads:[~2019-09-13 20:21 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-13 20:13 Markus Armbruster [this message]
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 01/16] scripts/git.orderfile: Match QAPI schema more precisely Markus Armbruster
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 02/16] qapi: Drop check_type()'s redundant parameter @allow_optional Markus Armbruster
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 03/16] qapi: Drop support for boxed alternate arguments Markus Armbruster
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 04/16] docs/devel/qapi-code-gen: Minor specification fixes Markus Armbruster
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 05/16] tests/qapi-schema: Demonstrate bad reporting of funny characters Markus Armbruster
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 06/16] qapi: Restrict strings to printable ASCII Markus Armbruster
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 07/16] qapi: Drop support for escape sequences other than \\ Markus Armbruster
2019-09-17 16:18 ` Eric Blake
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 08/16] qapi: Permit 'boxed' with empty type Markus Armbruster
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 09/16] qapi: Permit alternates with just one branch Markus Armbruster
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 10/16] qapi: Permit omitting all flat union branches Markus Armbruster
2019-09-17 16:20 ` Eric Blake
2019-09-23 11:46 ` Markus Armbruster
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 11/16] qapi: Adjust frontend errors to say enum value, not member Markus Armbruster
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 12/16] docs/devel/qapi-code-gen: Reorder sections for readability Markus Armbruster
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 13/16] docs/devel/qapi-code-gen: Rewrite compatibility considerations Markus Armbruster
2019-09-17 16:22 ` Eric Blake
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 14/16] docs/devel/qapi-code-gen: Rewrite introduction to schema Markus Armbruster
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 15/16] docs/devel/qapi-code-gen: Improve QAPI schema language doc Markus Armbruster
2019-09-13 20:13 ` [Qemu-devel] [PATCH v3 16/16] qapi: Tweak code to match docs/devel/qapi-code-gen.txt Markus Armbruster
2019-09-14 2:49 ` [Qemu-devel] [PATCH v3 00/16] qapi: Schema language cleanups & doc improvements no-reply
2019-09-17 16:31 ` Eric Blake
2019-09-23 11:49 ` Markus Armbruster
2019-09-23 18:29 ` Markus Armbruster
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190913201349.24332-1-armbru@redhat.com \
--to=armbru@redhat.com \
--cc=marcandre.lureau@redhat.com \
--cc=mdroth@linux.vnet.ibm.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).