From mboxrd@z Thu Jan  1 00:00:00 1970
Content-Type: multipart/mixed; boundary="===============5271084194445409390=="
MIME-Version: 1.0
From: Denis Kenzior <denkenz@gmail.com>
Subject: Re: [PATCH 6/6] hfp_hf_bluez5: Handle org.bluez.Profile1.Release()
Date: Mon, 19 Aug 2013 12:57:53 -0500
Message-ID: <52125CA1.5020203@gmail.com>
In-Reply-To: <1376619969-15080-7-git-send-email-vcgomes@gmail.com>
List-Id: <ofono.ofono.org>
To: ofono@ofono.org

--===============5271084194445409390==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

Hi Vinicius,

On 08/15/2013 09:26 PM, Vinicius Costa Gomes wrote:
> There's nothing we can do in this case, so we only reply with a
> success.
> ---
>   plugins/hfp_hf_bluez5.c | 4 +---
>   1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/plugins/hfp_hf_bluez5.c b/plugins/hfp_hf_bluez5.c
> index 7e98a45..391a74e 100644
> --- a/plugins/hfp_hf_bluez5.c
> +++ b/plugins/hfp_hf_bluez5.c
> @@ -564,9 +564,7 @@ static DBusMessage *profile_release(DBusConnection *c=
onn,
>   {
>   	DBG("Profile handler Release");
>
> -	return g_dbus_create_error(msg, BLUEZ_ERROR_INTERFACE
> -						".NotImplemented",
> -						"Implementation not provided");
> +	return dbus_message_new_method_return(msg);

To me it seems like this method is not properly labeled as NOREPLY =

inside the BlueZ API specification.  Sending a reply here seems pointless.

>   }
>
>   static DBusMessage *profile_cancel(DBusConnection *conn,
>

Regards,
-Denis

--===============5271084194445409390==--