Re: [Qemu-devel] [PULL 00/15] Audio 20190813 patches

[Peter Maydell]

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