Re: [PATCH BlueZ v1 06/12] core: Rename device_attach_attrib

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Luiz,

> On Mon, Dec 15, 2014 at 3:51 AM, Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx> wrote:
> Hi Arman,
>
> On Wed, Dec 10, 2014 at 1:00 AM, Arman Uguray <armansito@xxxxxxxxxxxx> wrote:
>> This patch renames the device_attach_attrib function to
>> device_attach_att_transport to remove ambiguities arise from the word
>> "attrib".
>> ---
>>  src/attrib-server.c | 2 +-
>>  src/device.c        | 4 ++--
>>  src/device.h        | 2 +-
>>  3 files changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/src/attrib-server.c b/src/attrib-server.c
>> index 6571577..e73850d 100644
>> --- a/src/attrib-server.c
>> +++ b/src/attrib-server.c
>> @@ -1300,7 +1300,7 @@ static void connect_event(GIOChannel *io, GError *gerr, void *user_data)
>>         if (!device)
>>                 return;
>>
>> -       device_attach_attrib(device, io);
>> +       device_attach_att_transport(device, io);
>>  }
>>
>>  static gboolean register_core_services(struct gatt_server *server)
>> diff --git a/src/device.c b/src/device.c
>> index bb80409..3bd3939 100644
>> --- a/src/device.c
>> +++ b/src/device.c
>> @@ -3637,7 +3637,7 @@ static void gatt_client_init(struct btd_device *device)
>>         DBG("gatt-client created");
>>  }
>>
>> -bool device_attach_attrib(struct btd_device *dev, GIOChannel *io)
>> +bool device_attach_att_transport(struct btd_device *dev, GIOChannel *io)
>>  {
>>         GError *gerr = NULL;
>>         GAttrib *attrib;
>> @@ -3725,7 +3725,7 @@ static void att_connect_cb(GIOChannel *io, GError *gerr, gpointer user_data)
>>                 goto done;
>>         }
>>
>> -       if (!device_attach_attrib(device, io))
>> +       if (!device_attach_att_transport(device, io))
>>                 goto done;
>>
>>         if (attcb->success)
>> diff --git a/src/device.h b/src/device.h
>> index 2e0473e..10c4951 100644
>> --- a/src/device.h
>> +++ b/src/device.h
>> @@ -69,7 +69,7 @@ struct gatt_primary *btd_device_get_primary(struct btd_device *device,
>>  GSList *btd_device_get_primaries(struct btd_device *device);
>>  void btd_device_gatt_set_service_changed(struct btd_device *device,
>>                                                 uint16_t start, uint16_t end);
>> -bool device_attach_attrib(struct btd_device *dev, GIOChannel *io);
>> +bool device_attach_att_transport(struct btd_device *dev, GIOChannel *io);
>
> Lets call device_attach_att and drop the transport suffix.
>
>
> --
> Luiz Augusto von Dentz

I forgot to include this in v2, so I'll upload v3 for this one patch.

-Arman
--
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




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux