RE: [PATCH] gatt-server: Fix mutlpile definition of same func

[Gowtham Anandha Babu <gowtham.ab@samsung.com>]

Hi Johan & Luiz,

> -----Original Message-----
> From: linux-bluetooth-owner@vger.kernel.org [mailto:linux-bluetooth-
> owner@vger.kernel.org] On Behalf Of Luiz Augusto von Dentz
> Sent: Friday, February 13, 2015 7:19 PM
> To: Gowtham Anandha Babu; linux-bluetooth@vger.kernel.org; Bharat
> Panda; cpgs@samsung.com
> Subject: Re: [PATCH] gatt-server: Fix mutlpile definition of same func
> 
> Hi Johan,
> 
> On Fri, Feb 13, 2015 at 1:33 PM, Johan Hedberg <johan.hedberg@gmail.com>
> wrote:
> > Hi,
> >
> > On Fri, Feb 06, 2015, Gowtham Anandha Babu wrote:
> >> --- a/src/shared/gatt-helpers.h
> >> +++ b/src/shared/gatt-helpers.h
> >> @@ -105,3 +105,5 @@ bool bt_gatt_read_by_type(struct bt_att *att,
> uint16_t start, uint16_t end,
> >>                                       bt_gatt_discovery_callback_t callback,
> >>                                       void *user_data,
> >>                                       bt_gatt_destroy_func_t
> >> destroy);
> >> +
> >> +void put_uuid_le(const bt_uuid_t *src, void *dst);
> >
> > gatt-helpers.h uses a namespace prefix of "bt_gatt" so you should at
> > the very least conform to that. The function name would then become e.g.
> > bt_gatt_put_uuid(). Besides that I don't have any objections to the
> > patch.
> 
> You are absolutely right, this function does not belong here, Ive send and
> patch moving it to lib/uuid.h where bt_uuid is implemented since it is given
> as parameter here.
> 
> @Gowtham, I hope you are paying attention to the outcome of these
> changes.
> 

I am about to submit a patch to address Johan comments (to add "bt_uuid" prefix to put_uuid_le func).
Anyway your patch optimizes the whole thing. Thanks !.

Since I am new to BlueZ, may be it will take some time get an overview of all.
Please feel free to correct my patches in future, if anything is wrong.

Regards,
Gowtham Anandha Babu

> 
> --
> Luiz Augusto von Dentz
> --
> To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
> the body of a message to majordomo@vger.kernel.org More majordomo
> info at  http://vger.kernel.org/majordomo-info.html