Hi Frédéric,

On 04/06/2012 08:17 AM, Frédéric Danis wrote:
> ---
>  plugins/bluetooth.h |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/plugins/bluetooth.h b/plugins/bluetooth.h
> index f81fc99..41ce112 100644
> --- a/plugins/bluetooth.h
> +++ b/plugins/bluetooth.h
> @@ -30,6 +30,7 @@
>  #define      BLUEZ_SERVICE_INTERFACE         BLUEZ_SERVICE ".Service"
>  #define      BLUEZ_TELEPHONY_INTERFACE       BLUEZ_SERVICE ".Telephony"
>  #define      BLUEZ_TELEPHONY_AGENT_INTERFACE BLUEZ_SERVICE ".TelephonyAgent"
> +#define      BLUEZ_TRANSPORT_INTERFACE       BLUEZ_SERVICE ".MediaTransport"
>  
>  #define DBUS_TIMEOUT 15
>  
> @@ -85,6 +86,9 @@ int bluetooth_sap_client_register(struct 
> bluetooth_sap_driver *sap,
>                                       struct ofono_modem *modem);
>  void bluetooth_sap_client_unregister(struct ofono_modem *modem);
>  
> +void *bluetooth_set_audio_management(void *em, const char *path,
> +                                                     guint16 features);
> +void bluetooth_free_audio_management(void *data);

I don't like passing around void pointers.  We should either try to
avoid this altogether or forward declare the struct.

>  int bluetooth_parse_newconnection_message(DBusMessage *msg, const char 
> **device,
>                                       const char **uuid, guint16 *version,
>                                       guint16 *features,

Regards,
-Denis
_______________________________________________
ofono mailing list
[email protected]
http://lists.ofono.org/listinfo/ofono

Reply via email to