On 2019-09-19 17:28, Eric Blake wrote:
On 9/19/19 10:15 AM, Peter Maydell wrote:
On Thu, 19 Sep 2019 at 09:38, Gerd Hoffmann <kra...@redhat.com> wrote:

The following changes since commit f8c3db33a5e863291182f8862ddf81618a7c6194:

   target/sparc: Switch to do_transaction_failed() hook (2019-09-17 12:01:00 
+0100)

are available in the Git repository at:

   git://git.kraxel.org/qemu tags/audio-20190919-pull-request

for you to fetch changes up to cf0c1c2aa32db5d658c3c797ad995a6d571bad96:

   audio: fix ALSA period-length typo in documentation (2019-09-19 10:32:48 
+0200)

----------------------------------------------------------------
audio: make mixeng optional.
audio: add surround sound support.
audio: documentation fixes.


Hi; I'm afraid this fails to build on OSX/FreeBSD/OpenBSD/Windows,
with format string issues:

/Users/pm215/src/qemu-for-merges/audio/wavaudio.c:50:15: error: format
specifies type 'size_t' (aka 'unsigned long') but the argument has
type 'int64_t' (aka 'long long') [-Werror,-Wformat]
               bytes, strerror(errno));
               ^~~~~

As long as you spin a v2, it's also worth fixing a typo I found in 15/26.


Thanks, fixed in v4, along with the compilation problem.

Reply via email to