On Mon, Sep 15, 2014 at 11:26:14AM -0500, Jeremy White wrote: > > Signed-off-by: Jeremy White <[email protected]> > --- > src/spiceqxl_audio.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/spiceqxl_audio.c b/src/spiceqxl_audio.c > index 7afc089..02859ee 100644 > --- a/src/spiceqxl_audio.c > +++ b/src/spiceqxl_audio.c > @@ -195,10 +195,10 @@ audio_thread_main (void *p) > struct audio_data data; > int freq = SPICE_INTERFACE_PLAYBACK_FREQ; > > + memset(&data, 0, sizeof(data));
Could this be written as struct audio_data data = { 0, }; or something
like that?
The memset(data.buffer, 0, data.buffer_bytes); call below becomes
redundant if I''m not mistaken.
Christophe
> for (i = 0; i < MAX_FIFOS; ++i)
> data.fifo_fds[i] = -1;
>
> - data.valid_bytes = data.fed = 0;
> #if SPICE_INTERFACE_PLAYBACK_MAJOR > 1 || SPICE_INTERFACE_PLAYBACK_MINOR >= 3
> freq = spice_server_get_best_playback_rate(&qxl->playback_sin);
> #endif
> --
> 1.7.10.4
>
> _______________________________________________
> Spice-devel mailing list
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
pgpMB3E20_iXR.pgp
Description: PGP signature
_______________________________________________ Spice-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/spice-devel
