On 06/19/2013 05:40 PM, Tanu Kaskinen wrote:
This is safer and more convenient for subclasses. --- src/pulsecore/object.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/src/pulsecore/object.c b/src/pulsecore/object.c index 0dc8198..b6fbe90 100644 --- a/src/pulsecore/object.c +++ b/src/pulsecore/object.c @@ -40,7 +40,7 @@ pa_object *pa_object_new_internal(size_t size, const char *type_id, pa_bool_t (* pa_assert(check_type(type_id)); pa_assert(check_type(pa_object_type_id)); - o = pa_xmalloc(size); + o = pa_xmalloc0(size); PA_REFCNT_INIT(o); o->type_id = type_id; o->free = pa_object_free;
I think this is a good change in general. It gives the opportunity for several cleanups in object implementations where things are explicitly set to zero, because with this patch only I assume a lot of things will be zeroed twice.
-- David Henningsson, Canonical Ltd. https://launchpad.net/~diwic _______________________________________________ pulseaudio-discuss mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
