On Tue, 2013-09-24 at 19:45 -0300, [email protected] wrote:
> @@ -371,11 +398,8 @@ static void adapter_remove_all(pa_bluetooth_discovery 
> *y) {
>  
>      /* When this function is called all devices have already been freed */
>  
> -    while ((a = pa_hashmap_steal_first(y->adapters))) {
> -        pa_xfree(a->path);
> -        pa_xfree(a->address);
> -        pa_xfree(a);
> -    }
> +    while ((a = pa_hashmap_steal_first(y->adapters)))
> +        adapter_free(a);

pa_hashmap_remove_all() can be used here (the hashmap initialization
needs to be changed so that adapter_free is passed to
pa_hashmap_new_full()). Actually, the whole adapter_remove_all()
function is now redundant, because it's effectively just an alias for
pa_hashmap_remove_all().

-- 
Tanu

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

Reply via email to