On Fri, Sep 19, 2014 at 09:09:43AM -0500, Jeremy White wrote:
> Hey Christophe,
>
> Thanks for the patch review.
>
> >>+ memset(&data, 0, sizeof(data));
> >
> >Could this be written as struct audio_data data = { 0, }; or something
> >like that?
>
> Sure.
>
> Note, though, that afaict, memset(...0...) seems more prevalent in the spice
> code than the universal zero initializer.memset is fine with me. I'm also a bit curious about the warnings you got, but I guess you no longer have these around. > > But I need to break my grumpy K&R edition 1 only ways, so it'll be a good > exercise for me <grin>. > > >The memset(data.buffer, 0, data.buffer_bytes); call below becomes > >redundant if I''m not mistaken. > > No, that's malloc'd. (I suppose it should probably be a calloc, but I'd > feel that would be out of scope for this patch). Indeed it is, my bad. ACK patch. Christophe
pgpVK1uDnAIHe.pgp
Description: PGP signature
_______________________________________________ Spice-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/spice-devel
