On Tue, 2014-02-04 at 19:03 -0300, [email protected] wrote:
> +void hf_audio_agent_done(hf_audio_agent_data *data) {
> +    hf_audio_agent_data *hfdata = data;

What's the point of this extra variable? Why not use data directly?

> +
> +    pa_assert(hfdata);
> +
> +    if (hfdata->connection) {
> +        
> dbus_connection_unregister_object_path(pa_dbus_connection_get(hfdata->connection),
>  HF_AUDIO_AGENT_PATH);
> +
> +        pa_dbus_connection_unref(hfdata->connection);
> +    }
> +
> +    pa_xfree(hfdata);
> +}

-- 
Tanu

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

Reply via email to