All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Eric Blake <eblake@redhat.com>
Cc: qemu-devel@nongnu.org, armbru@redhat.com
Subject: Re: [Qemu-devel] [PATCH 00/36] Rip out dynamic JSON parsing
Date: Wed, 30 Nov 2016 15:42:25 -0500 (EST)	[thread overview]
Message-ID: <772788740.857401.1480538545524.JavaMail.zimbra@redhat.com> (raw)
In-Reply-To: <aca3ec07-fd6a-90c4-0eb3-b75f34e4a6fa@redhat.com>



----- Original Message -----
> From: "Eric Blake" <eblake@redhat.com>
> To: qemu-devel@nongnu.org
> Cc: pbonzini@redhat.com, armbru@redhat.com
> Sent: Wednesday, November 30, 2016 8:50:05 PM
> Subject: Re: [Qemu-devel] [PATCH 00/36] Rip out dynamic JSON parsing
> 
> On 11/30/2016 01:44 PM, Eric Blake wrote:
> > Followup series to conversation about PRId64 and MacOS:
> > https://lists.gnu.org/archive/html/qemu-devel/2016-11/msg04226.html
> > 
> 
> And I meant to add:
> 
> Available as a tag at:
> git fetch git://repo.or.cz/qemu/ericb.git qapi-dynamic-json-v1
> 
> on top of Markus' qapi-next branch (but presumably will apply without
> much effort on top of the final 2.8 release, since this is 2.9 material)

I don't see the point, honestly.  It's harder to use and hardly
removes any code (the balance is -160 lines, but about 140 of them
come from the parser's test suite).

Using the functionality only in qtest is fine, but it is damn useful
there.  If we want to remove the % escaping, let's rewrite the qtests
in Python as they probably should have been since the beginning.

Paolo

      reply	other threads:[~2016-11-30 20:42 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-30 19:44 [Qemu-devel] [PATCH 00/36] Rip out dynamic JSON parsing Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 01/36] pci: Use struct instead of QDict to pass back parameters Eric Blake
2016-11-30 21:15   ` Michael S. Tsirkin
2016-11-30 19:44 ` [Qemu-devel] [PATCH 02/36] qdict: Add convenience helpers for wrapped puts Eric Blake
2016-11-30 19:44   ` Eric Blake
2016-12-09 14:06   ` [Qemu-devel] " Alberto Garcia
2016-12-09 14:06     ` Alberto Garcia
2016-11-30 19:44 ` [Qemu-devel] [PATCH 03/36] qlist: Add convenience helpers for wrapped appends Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 04/36] qmp-event: Avoid dynamic JSON Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 05/36] qmp-dispatch: " Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 06/36] qobject-input-visitor: Avoid dynamic JSON in tests Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 07/36] fdc-test: Avoid deprecated 'change' command Eric Blake
2016-11-30 20:58   ` John Snow
2016-11-30 19:44 ` [Qemu-devel] [PATCH 08/36] test-qga: Actually test 0xff sync bytes Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 09/36] qtest: Add a new helper qmp_cmd() and friends Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 10/36] qtest: Avoid dynamic JSON in libqtest Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 11/36] qapi: Add QAPI_TO_QOBJECT() convenience macro Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 12/36] nbd: Use simpler QAPI_TO_QOBJECT() Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 13/36] nfs: " Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 14/36] qapi: " Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 15/36] blockdev: " Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 16/36] qapi: Promote blockdev-change-medium arguments to QAPI type Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 17/36] qtest: Avoid dynamic JSON in ahci-test Eric Blake
2016-11-30 21:02   ` John Snow
2016-11-30 21:19     ` Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 18/36] qtest: Avoid dynamic JSON in fdc-test Eric Blake
2016-11-30 21:14   ` John Snow
2016-11-30 19:44 ` [Qemu-devel] [PATCH 19/36] qtest: Change qmp_discard_response() to drop varargs Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 20/36] qtest: Avoid dynamic JSON in device-introspect-test Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 21/36] qtest: Avoid dynamic JSON in tmp105-test Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 22/36] qtest: Avoid dynamic JSON in pc-cpu-test Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 23/36] qtest: Avoid dynamic JSON in virtio-blk-test Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 24/36] qtest: Drop unused qmp_fdv() Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 25/36] qtest: Change qmp_fd_send() to drop varags Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 26/36] qtest: Drop unused qtest_qmp_async() Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 27/36] qtest: Avoid dynamic JSON in qmp_cmd() Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 28/36] qapi: Factor out JSON string escaping Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 29/36] qapi: Add qstring_append_printf() Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 30/36] qtest: Avoid dynamic JSON in qmp_fd_sendv() Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 31/36] qtest: Document calling conventions Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 32/36] qtest: Avoid dynamic JSON in qom-test Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 33/36] qtest: Avoid dynamic JSON in test-x86-cpuid-compat Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 34/36] qapi: Rip out dynamic JSON parser frontend Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 35/36] qapi: Rip out dynamic JSON parser escape sequence support Eric Blake
2016-11-30 19:44 ` [Qemu-devel] [PATCH 36/36] qapi: Rip out dynamic JSON parser backend Eric Blake
2016-11-30 19:50 ` [Qemu-devel] [PATCH 00/36] Rip out dynamic JSON parsing Eric Blake
2016-11-30 20:42   ` Paolo Bonzini [this message]

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=772788740.857401.1480538545524.JavaMail.zimbra@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=armbru@redhat.com \
    --cc=eblake@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.