All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Blake <eblake@redhat.com>
To: "Kővágó, Zoltán" <dirty.ice.hu@gmail.com>, qemu-devel@nongnu.org
Cc: Markus Armbruster <armbru@redhat.com>,
	Michael Roth <mdroth@linux.vnet.ibm.com>,
	Gerd Hoffmann <kraxel@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 01/51] qapi: support implicit structs in OptsVisitor
Date: Thu, 14 Jan 2016 09:43:59 -0700	[thread overview]
Message-ID: <5697D04F.4090203@redhat.com> (raw)
In-Reply-To: <fa23c5c5ee69322e77aaf1a13ac5cdc312efe187.1452776884.git.DirtY.iCE.hu@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 603 bytes --]

On 01/14/2016 06:45 AM, Kővágó, Zoltán wrote:
> They are required for flat unions (you still have to allocate the
> structs).
> 
> Signed-off-by: Kővágó, Zoltán <DirtY.iCE.hu@gmail.com>
> ---
>  qapi/opts-visitor.c | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)

I basically have a modified version of this patch already in my pending
work:
https://lists.gnu.org/archive/html/qemu-devel/2015-12/msg04393.html

but rebased on top of other pending qapi changes.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 604 bytes --]

  reply	other threads:[~2016-01-14 16:44 UTC|newest]

Thread overview: 56+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-14 13:45 [Qemu-devel] [PATCH 00/51] audio 5.1 patches Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 01/51] qapi: support implicit structs in OptsVisitor Kővágó, Zoltán
2016-01-14 16:43   ` Eric Blake [this message]
2016-01-14 13:45 ` [Qemu-devel] [PATCH 02/51] qapi: support nested " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 03/51] qapi: qapi for audio backends Kővágó, Zoltán
2016-01-14 16:51   ` Eric Blake
2016-01-15 13:49     ` Gerd Hoffmann
2016-01-14 13:45 ` [Qemu-devel] [PATCH 04/51] audio: use qapi AudioFormat instead of audfmt_e Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 05/51] audio: -audiodev command line option: documentation Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 06/51] audio: -audiodev command line option basic implementation Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 07/51] alsaaudio: port to -audiodev config Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 08/51] coreaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 09/51] dsoundaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 10/51] noaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 11/51] ossaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 12/51] paaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 13/51] sdlaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 14/51] spiceaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 15/51] wavaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 16/51] audio: -audiodev command line option: cleanup Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 17/51] audio: reduce glob_audio_state usage Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 18/51] audio: basic support for multi backend audio Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 19/51] audio: add audiodev properties to frontends Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 20/51] audio: audiodev= parameters no longer optional when -audiodev present Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 21/51] paaudio: do not move stream when sink/source name is specified Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 22/51] paaudio: properly disconnect streams in fini_* Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 23/51] audio: remove audio_MIN, audio_MAX Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 24/51] audio: do not run each backend in audio_run Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 25/51] paaudio: fix playback glitches Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 26/51] audio: remove read and write pcm_ops Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 27/51] audio: use size_t where makes sense Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 28/51] audio: api for mixeng code free backends Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 29/51] alsaaudio: port to the new audio backend api Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 30/51] coreaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 31/51] noaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 32/51] ossaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 33/51] paaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 34/51] sdlaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 36/51] wavaudio: " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 37/51] audio: remove remains of the old " Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 38/51] audio: unify input and output mixeng buffer management Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 39/51] audio: remove hw->samples, buffer_size_in/out pcm_ops Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 40/51] audio: common rate control code for timer based outputs Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 41/51] audio: split ctl_* functions into enable_* and volume_* Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 42/51] audio: add mixeng option (documentation) Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 43/51] audio: make mixeng optional Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 44/51] paaudio: get/put_buffer functions Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 45/51] audio: support more than two channels in volume setting Kővágó, Zoltán
2016-01-14 13:45 ` [Qemu-devel] [PATCH 46/51] audio: replace shift in audio_pcm_info with bytes_per_frame Kővágó, Zoltán
2016-01-14 13:46 ` [Qemu-devel] [PATCH 47/51] audio: basic support for multichannel audio Kővágó, Zoltán
2016-01-14 13:46 ` [Qemu-devel] [PATCH 48/51] paaudio: channel-map option Kővágó, Zoltán
2016-01-14 13:46 ` [Qemu-devel] [PATCH 49/51] usb-audio: do not count on avail bytes actually available Kővágó, Zoltán
2016-01-14 13:46 ` [Qemu-devel] [PATCH 51/51] usbaudio: change playback counters to 64 bit Kővágó, Zoltán
2016-01-14 14:34 ` [Qemu-devel] [PATCH 00/51] audio 5.1 patches Victor Toso
2016-01-14 15:30   ` Kővágó Zoltán
2016-01-15 13:51 ` Gerd Hoffmann

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=5697D04F.4090203@redhat.com \
    --to=eblake@redhat.com \
    --cc=armbru@redhat.com \
    --cc=dirty.ice.hu@gmail.com \
    --cc=kraxel@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 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.