From: Peter Maydell <peter.maydell@linaro.org>
To: qemu-devel@nongnu.org
Cc: qemu-trivial@nongnu.org, "Anthony Liguori" <aliguori@us.ibm.com>,
"John Rigby" <john.rigby@linaro.org>,
"Andreas Färber" <afaerber@suse.de>,
patches@linaro.org
Subject: [Qemu-devel] [PATCH] qapi-schema.json: Reformat TargetType enum to one-per-line
Date: Mon, 20 May 2013 17:21:24 +0100 [thread overview]
Message-ID: <1369066884-431-1-git-send-email-peter.maydell@linaro.org> (raw)
Reformat the qapi-schema TargetType enumeration so that it has just
one target architecture name per line. This allows patches for
adding new targets to just add a single line, rather than having
to reformat most of the list (resulting in a hard-to-check diff).
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
d15a9c23 is an example of what you get otherwise.
I would much prefer it if we autogenerated this list so you didn't
need to change this file at all to add a new target, but Anthony
is against that; so this is at least an improvement.
qapi-schema.json | 30 ++++++++++++++++++++++++++----
1 file changed, 26 insertions(+), 4 deletions(-)
diff --git a/qapi-schema.json b/qapi-schema.json
index 199744a..a8d361e 100644
--- a/qapi-schema.json
+++ b/qapi-schema.json
@@ -3018,10 +3018,32 @@
# Since: 1.2.0
##
{ 'enum': 'TargetType',
- 'data': [ 'alpha', 'arm', 'cris', 'i386', 'lm32', 'm68k', 'microblazeel',
- 'microblaze', 'mips64el', 'mips64', 'mipsel', 'mips', 'moxie',
- 'or32', 'ppc64', 'ppcemb', 'ppc', 's390x', 'sh4eb', 'sh4',
- 'sparc64', 'sparc', 'unicore32', 'x86_64', 'xtensaeb', 'xtensa' ] }
+ 'data': [ 'alpha',
+ 'arm',
+ 'cris',
+ 'i386',
+ 'lm32',
+ 'm68k',
+ 'microblazeel',
+ 'microblaze',
+ 'mips64el',
+ 'mips64',
+ 'mipsel',
+ 'mips',
+ 'moxie',
+ 'or32',
+ 'ppc64',
+ 'ppcemb',
+ 'ppc',
+ 's390x',
+ 'sh4eb',
+ 'sh4',
+ 'sparc64',
+ 'sparc',
+ 'unicore32',
+ 'x86_64',
+ 'xtensaeb',
+ 'xtensa' ] }
##
# @TargetInfo:
--
1.7.9.5
next reply other threads:[~2013-05-20 16:32 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-20 16:21 Peter Maydell [this message]
2013-05-20 16:35 ` [Qemu-devel] [PATCH] qapi-schema.json: Reformat TargetType enum to one-per-line Paolo Bonzini
2013-05-20 16:38 ` Peter Maydell
2013-05-20 17:14 ` Paolo Bonzini
2013-05-22 13:15 ` Anthony Liguori
2013-05-22 13:50 ` Andreas Färber
2013-05-22 14:28 ` Anthony Liguori
2013-05-22 14:34 ` [Qemu-devel] New targets (was: [PATCH] qapi-schema.json: Reformat TargetType enum to one-per-line) Andreas Färber
2013-05-22 14:48 ` Anthony Liguori
2013-05-22 15:33 ` Peter Maydell
2013-05-22 16:10 ` Anthony Liguori
2013-05-22 13:51 ` [Qemu-devel] [PATCH] qapi-schema.json: Reformat TargetType enum to one-per-line Peter Maydell
2013-05-22 14:29 ` Anthony Liguori
2013-05-22 14:38 ` Paolo Bonzini
2013-05-24 21:38 ` [Qemu-devel] [libvirt] " Eric Blake
2013-05-25 9:18 ` Peter Maydell
2013-05-25 12:31 ` Andreas Färber
2013-05-20 16:41 ` [Qemu-devel] " Eric Blake
2013-05-20 16:47 ` Peter Maydell
2013-05-20 16:57 ` Eric Blake
2013-05-20 17:05 ` Paolo Bonzini
2013-05-22 13:12 ` Anthony Liguori
2013-05-22 13:38 ` Peter Maydell
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=1369066884-431-1-git-send-email-peter.maydell@linaro.org \
--to=peter.maydell@linaro.org \
--cc=afaerber@suse.de \
--cc=aliguori@us.ibm.com \
--cc=john.rigby@linaro.org \
--cc=patches@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-trivial@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).