Hi Lucas, On Mon, Jun 11, 2012 at 6:16 PM, Lucas De Marchi <lucas.demarchi@xxxxxxxxxxxxxx> wrote: >> + if (transport->destroy != NULL) >> + transport->destroy(transport); > > IMO it would be better to pass transport->data here and eliminate > "transport->data" assignment above in the destroy functions. That was my idea, but since destroy_headset need access to the device it was just simpler to just pass the transport itself. Not a big deal IMO since this in the same file, but I guess I will just store the device inside the headset_transport. -- Luiz Augusto von Dentz -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
