[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/15] Audio 20190813 patches
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PULL 00/15] Audio 20190813 patches |
Date: |
Fri, 16 Aug 2019 11:10:26 +0100 |
On Tue, 13 Aug 2019 at 12:20, Gerd Hoffmann <address@hidden> wrote:
>
> The following changes since commit 5e7bcdcfe69ce0fad66012b2cfb2035003c37eef:
>
> display/bochs: fix pcie support (2019-08-12 16:36:41 +0100)
>
> are available in the Git repository at:
>
> git://git.kraxel.org/qemu tags/audio-20190813-pull-request
>
> for you to fetch changes up to 7f6f83e89de6967db66a87f7ceb7be3c02a24d56:
>
> audio: Add missing fall through comments (2019-08-13 12:17:57 +0200)
>
> ----------------------------------------------------------------
> audio: second batch of -audiodev support, adding support for multiple
> backends.
>
> ----------------------------------------------------------------
Hi; this fails the Windows build:
/home/petmay01/qemu-for-merges/audio/dsoundaudio.c: In function
'dsound_run_out':
/home/petmay01/qemu-for-merges/audio/dsoundaudio.c:537:33: error:
comparison of unsigned expression < 0 is always false
[-Werror=type-limits]
if (audio_bug(__func__, len < 0 || len > bufsize)) {
^
In file included from /home/petmay01/qemu-for-merges/audio/dsoundaudio.c:33:0:
/home/petmay01/qemu-for-merges/audio/dsoundaudio.c:538:16: error:
format '%d' expects argument of type 'int', but argument 3 has type
'size_t {aka long long unsigned int}' [-Werror=format=]
dolog ("len=%d bufsize=%d old_pos=%ld ppos=%ld\n",
^
/home/petmay01/qemu-for-merges/audio/audio_int.h:232:44: note: in
definition of macro 'dolog'
#define dolog(fmt, ...) AUD_log(AUDIO_CAP, fmt, ## __VA_ARGS__)
^
/home/petmay01/qemu-for-merges/audio/dsoundaudio.c:538:16: error:
format '%d' expects argument of type 'int', but argument 4 has type
'size_t {aka long long unsigned int}' [-Werror=format=]
dolog ("len=%d bufsize=%d old_pos=%ld ppos=%ld\n",
^
/home/petmay01/qemu-for-merges/audio/audio_int.h:232:44: note: in
definition of macro 'dolog'
#define dolog(fmt, ...) AUD_log(AUDIO_CAP, fmt, ## __VA_ARGS__)
^
cc1: all warnings being treated as errors
/home/petmay01/qemu-for-merges/rules.mak:69: recipe for target
'audio/dsoundaudio.o' failed
thanks
-- PMM
- [Qemu-devel] [PULL 02/15] audio: basic support for multi backend audio, (continued)
- [Qemu-devel] [PULL 02/15] audio: basic support for multi backend audio, Gerd Hoffmann, 2019/08/13
- [Qemu-devel] [PULL 08/15] paaudio: properly disconnect streams in fini_*, Gerd Hoffmann, 2019/08/13
- [Qemu-devel] [PULL 13/15] audio: use size_t where makes sense, Gerd Hoffmann, 2019/08/13
- [Qemu-devel] [PULL 06/15] audio: audiodev= parameters no longer optional when -audiodev present, Gerd Hoffmann, 2019/08/13
- [Qemu-devel] [PULL 15/15] audio: Add missing fall through comments, Gerd Hoffmann, 2019/08/13
- [Qemu-devel] [PULL 12/15] audio: remove read and write pcm_ops, Gerd Hoffmann, 2019/08/13
- [Qemu-devel] [PULL 10/15] audio: do not run each backend in audio_run, Gerd Hoffmann, 2019/08/13
- [Qemu-devel] [PULL 01/15] audio: reduce glob_audio_state usage, Gerd Hoffmann, 2019/08/13
- Re: [Qemu-devel] [PULL 00/15] Audio 20190813 patches, Peter Maydell, 2019/08/13
- Re: [Qemu-devel] [PULL 00/15] Audio 20190813 patches,
Peter Maydell <=