[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 01/14] qapi: qapi for audio backends
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH v6 01/14] qapi: qapi for audio backends |
Date: |
Tue, 12 Mar 2019 08:12:55 +0100 |
User-agent: |
NeoMutt/20180716 |
Hi,
> >> +# @buffer-length: the buffer length in microseconds
> >
> > The name buffer-length suggests bytes rather than microseconds. That's
> > why I suggested @buffer-capacity. Matter of taste, up to you and Gerd.
>
> @period-length has the same issue. For me @buffer-capacity and
> @period-capacity feels a bit weird. I'm not 100% familiar with qapi
> conventions, but for me 'size' is what's usually in bytes, and 'length'
> can be something else (number of characters in a string, length of a
> sound file, ...).
Agree here. "size" is linked to "bytes" in my head too, thinking of the
"sizeof()" operator or DEFINE_PROP_SIZE() for example.
Whereas "length" can be bytes (strlen) but can be other units too
(wcslen).
Pull request sent.
thanks,
Gerd
- [Qemu-devel] [PATCH v6 00/14] Audio patches, Kővágó, Zoltán, 2019/03/08
- [Qemu-devel] [PATCH v6 03/14] audio: -audiodev command line option: documentation, Kővágó, Zoltán, 2019/03/08
- [Qemu-devel] [PATCH v6 02/14] audio: use qapi AudioFormat instead of audfmt_e, Kővágó, Zoltán, 2019/03/08
- [Qemu-devel] [PATCH v6 08/14] noaudio: port to -audiodev config, Kővágó, Zoltán, 2019/03/08
- [Qemu-devel] [PATCH v6 07/14] dsoundaudio: port to -audiodev config, Kővágó, Zoltán, 2019/03/08
- [Qemu-devel] [PATCH v6 06/14] coreaudio: port to -audiodev config, Kővágó, Zoltán, 2019/03/08
- [Qemu-devel] [PATCH v6 05/14] alsaaudio: port to -audiodev config, Kővágó, Zoltán, 2019/03/08
- [Qemu-devel] [PATCH v6 11/14] sdlaudio: port to -audiodev config, Kővágó, Zoltán, 2019/03/08
- [Qemu-devel] [PATCH v6 09/14] ossaudio: port to -audiodev config, Kővágó, Zoltán, 2019/03/08
- [Qemu-devel] [PATCH v6 04/14] audio: -audiodev command line option basic implementation, Kővágó, Zoltán, 2019/03/08
- [Qemu-devel] [PATCH v6 10/14] paaudio: port to -audiodev config, Kővágó, Zoltán, 2019/03/08