qemu-devel
[Top][All Lists]
Advanced

[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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]