On Thu, 2012-03-29 at 12:03 +0300, Tanu Kaskinen wrote:
> Coverity warned that r was possibly being used without
> initialization. The fix is to add pa_assert_not_reached()
> to handle the case where the address family is invalid. I
> also felt that using switch would be more appropriate here
> than using ifs and else-ifs, which makes this patch a bit
> larger. I also changed the memset() calls to pa_zero().
> ---

Looks good.

-- Arun

_______________________________________________
pulseaudio-discuss mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss

Reply via email to