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