[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 01/16] scripts/git.orderfile: Match QAPI sche
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 01/16] scripts/git.orderfile: Match QAPI schema more precisely |
Date: |
Fri, 13 Sep 2019 16:14:05 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 9/10/19 1:37 AM, Markus Armbruster wrote:
>> Pattern *.json also matches the tests/qapi-schema/*.json. Separates
>> them from the tests/qapi-schema/*.{err,exit,out} in diffs. I hate
>> that. Change the pattern to match just the "real" QAPI schemata.
>>
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
>> scripts/git.orderfile | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>
> Should we ever do a mass rename of *.json into *.qapi (since our qapi
> files are NOT quite JSON)? But that doesn't change the viability of
> this patch.
Naming them .json is misleading. We're closer to Python than to JSON.
Many .json files ask Emacs for Python mode, because that works better
than JavaScript mode.
Options:
* Rename to .qapi. The files that don't ask for Python mode now get
Fundamental mode instead of JavaScript mode. Minor regression.
* Rename to .py, and drop the lines asking for Python mode. Still
misleading, but less so.
* Likewise, and tweak the syntax so it's actually Python.
* Switch to some less egregiously ill-suited syntax. TOML, perhaps.
* Do nothing.
Right now, I'd prefer to do nothing.
- [Qemu-devel] [PATCH v2 05/16] tests/qapi-schema: Demonstrate bad reporting of funny characters, (continued)
- [Qemu-devel] [PATCH v2 05/16] tests/qapi-schema: Demonstrate bad reporting of funny characters, Markus Armbruster, 2019/09/10
- [Qemu-devel] [PATCH v2 02/16] qapi: Drop check_type()'s redundant parameter @allow_optional, Markus Armbruster, 2019/09/10
- [Qemu-devel] [PATCH v2 03/16] qapi: Drop support for boxed alternate arguments, Markus Armbruster, 2019/09/10
- [Qemu-devel] [PATCH v2 01/16] scripts/git.orderfile: Match QAPI schema more precisely, Markus Armbruster, 2019/09/10
- [Qemu-devel] [PATCH v2 04/16] docs/devel/qapi-code-gen: Minor specification fixes, Markus Armbruster, 2019/09/10
- [Qemu-devel] [PATCH v2 11/16] qapi: Adjust frontend errors to say enum value, not member, Markus Armbruster, 2019/09/10
- [Qemu-devel] [PATCH v2 10/16] qapi: Permit omitting all flat union branches, Markus Armbruster, 2019/09/10
- [Qemu-devel] [PATCH v2 06/16] qapi: Restrict strings to printable ASCII, Markus Armbruster, 2019/09/10