Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1MVYai-0003dX-SY
	for qemu-devel@nongnu.org; Mon, 27 Jul 2009 18:22:44 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1MVYad-0003c3-Cm
	for qemu-devel@nongnu.org; Mon, 27 Jul 2009 18:22:43 -0400
Received: from [199.232.76.173] (port=37780 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1MVYad-0003bx-5Y
	for qemu-devel@nongnu.org; Mon, 27 Jul 2009 18:22:39 -0400
Received: from mx2.redhat.com ([66.187.237.31]:45236)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <quintela@redhat.com>) id 1MVYab-0004v6-PB
	for qemu-devel@nongnu.org; Mon, 27 Jul 2009 18:22:38 -0400
In-Reply-To: <Pine.LNX.4.64.0907280107590.3551@linmac.oyster.ru> (malc's
	message of "Tue\, 28 Jul 2009 01\:10\:19 +0400 \(MSD\)")
References: <cover.1248726105.git.quintela@redhat.com>
	<b4fdbc11fa13fab7899b166b64156be0ba743e64.1248726105.git.quintela@redhat.com>
	<Pine.LNX.4.64.0907280107590.3551@linmac.oyster.ru>
From: Juan Quintela <quintela@redhat.com>
Date: Tue, 28 Jul 2009 00:20:30 +0200
Message-ID: <m3iqhdsr81.fsf@neno.mitica>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Subject: [Qemu-devel] Re: [PATCH 3/4] Use C99 initializers for audio_option
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: malc <av1474@comtv.ru>
Cc: aliguori@us.ibm.com, qemu-devel@nongnu.org

malc <av1474@comtv.ru> wrote:
> On Mon, 27 Jul 2009, Juan Quintela wrote:
>
> I'd prefer:
>
>        { .name  = "DAC_SIZE_IN_USEC",
>          .tag   = AUD_OPT_BOOL,
>          .valp  = &conf.size_in_usec_out,
>          .descr = "DAC period/buffer size in microseconds (otherwise in frames)" },
>        { ...
>
> And likewise elsewhere.

I tried to be consistent with the qdev patches, they add all its "arrays
of structures" this way (it is one of the bigger users of the feature
that I saw).

Later, Juan.