Linux bluetooth development
 help / color / mirror / Atom feed
* [PATCH] advertising: Configure discoverable flag based on adapter settings
@ 2018-02-06 10:43 Szymon Janc
  2018-02-07  1:24 ` ERAMOTO Masaya
  0 siblings, 1 reply; 5+ messages in thread
From: Szymon Janc @ 2018-02-06 10:43 UTC (permalink / raw)
  To: linux-bluetooth; +Cc: Szymon Janc

Until now advertising as peripheral was always setting General Discoverable
flag set. With this patch this is set based on adapter discoverable setting.
---
 src/adapter.c     |  9 +++++++++
 src/adapter.h     |  1 +
 src/advertising.c | 19 +++++++++++++++++--
 src/advertising.h |  1 +
 4 files changed, 28 insertions(+), 2 deletions(-)

diff --git a/src/adapter.c b/src/adapter.c
index 50cf46b11..5070c0f09 100644
--- a/src/adapter.c
+++ b/src/adapter.c
@@ -543,6 +543,7 @@ static void settings_changed(struct btd_adapter *adapter, uint32_t settings)
 		g_dbus_emit_property_changed(dbus_conn, adapter->path,
 					ADAPTER_INTERFACE, "Discoverable");
 		store_adapter_info(adapter);
+		btd_adv_manager_refresh(adapter->adv_manager);
 	}
 
 	if (changed_mask & MGMT_SETTING_BONDABLE) {
@@ -4108,6 +4109,14 @@ bool btd_adapter_get_connectable(struct btd_adapter *adapter)
 	return false;
 }
 
+bool btd_adapter_get_discoverable(struct btd_adapter *adapter)
+{
+	if (adapter->current_settings & MGMT_SETTING_DISCOVERABLE)
+		return true;
+
+	return false;
+}
+
 struct btd_gatt_database *btd_adapter_get_database(struct btd_adapter *adapter)
 {
 	if (!adapter)
diff --git a/src/adapter.h b/src/adapter.h
index a85327cd1..e619a5be9 100644
--- a/src/adapter.h
+++ b/src/adapter.h
@@ -74,6 +74,7 @@ void adapter_foreach(adapter_cb func, gpointer user_data);
 bool btd_adapter_get_pairable(struct btd_adapter *adapter);
 bool btd_adapter_get_powered(struct btd_adapter *adapter);
 bool btd_adapter_get_connectable(struct btd_adapter *adapter);
+bool btd_adapter_get_discoverable(struct btd_adapter *adapter);
 
 struct btd_gatt_database *btd_adapter_get_database(struct btd_adapter *adapter);
 
diff --git a/src/advertising.c b/src/advertising.c
index 94a8c4050..637445189 100644
--- a/src/advertising.c
+++ b/src/advertising.c
@@ -622,8 +622,12 @@ static int refresh_adv(struct btd_adv_client *client, mgmt_request_func_t func)
 
 	DBG("Refreshing advertisement: %s", client->path);
 
-	if (client->type == AD_TYPE_PERIPHERAL)
-		flags = MGMT_ADV_FLAG_CONNECTABLE | MGMT_ADV_FLAG_DISCOV;
+	if (client->type == AD_TYPE_PERIPHERAL) {
+		flags = MGMT_ADV_FLAG_CONNECTABLE;
+
+		if (btd_adapter_get_discoverable(client->manager->adapter))
+			flags |= MGMT_ADV_FLAG_DISCOV;
+	}
 
 	flags |= client->flags;
 
@@ -1104,3 +1108,14 @@ void btd_adv_manager_destroy(struct btd_adv_manager *manager)
 
 	manager_destroy(manager);
 }
+void btd_adv_manager_refresh(struct btd_adv_manager *manager)
+{
+	const struct queue_entry *entry;
+
+	for (entry = queue_get_entries(manager->clients); entry;
+							entry = entry->next) {
+		struct btd_adv_client *client = entry->data;
+
+		refresh_adv(client, NULL);
+	}
+}
diff --git a/src/advertising.h b/src/advertising.h
index b783cf075..d98c4b864 100644
--- a/src/advertising.h
+++ b/src/advertising.h
@@ -22,3 +22,4 @@ struct btd_adv_manager;
 
 struct btd_adv_manager *btd_adv_manager_new(struct btd_adapter *adapter);
 void btd_adv_manager_destroy(struct btd_adv_manager *manager);
+void btd_adv_manager_refresh(struct btd_adv_manager *manager);
-- 
2.14.3


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] advertising: Configure discoverable flag based on adapter settings
  2018-02-06 10:43 [PATCH] advertising: Configure discoverable flag based on adapter settings Szymon Janc
@ 2018-02-07  1:24 ` ERAMOTO Masaya
  2018-02-07  8:46   ` Szymon Janc
  0 siblings, 1 reply; 5+ messages in thread
From: ERAMOTO Masaya @ 2018-02-07  1:24 UTC (permalink / raw)
  To: Szymon Janc; +Cc: linux-bluetooth

Hi Szymon,

On 02/06/2018 07:43 PM, Szymon Janc wrote:
> Until now advertising as peripheral was always setting General Discoverable
> flag set. With this patch this is set based on adapter discoverable setting.
> ---
>  src/adapter.c     |  9 +++++++++
>  src/adapter.h     |  1 +
>  src/advertising.c | 19 +++++++++++++++++--
>  src/advertising.h |  1 +
>  4 files changed, 28 insertions(+), 2 deletions(-)
> 
> diff --git a/src/adapter.c b/src/adapter.c
> index 50cf46b11..5070c0f09 100644
> --- a/src/adapter.c
> +++ b/src/adapter.c
> @@ -543,6 +543,7 @@ static void settings_changed(struct btd_adapter *adapter, uint32_t settings)
>  		g_dbus_emit_property_changed(dbus_conn, adapter->path,
>  					ADAPTER_INTERFACE, "Discoverable");
>  		store_adapter_info(adapter);
> +		btd_adv_manager_refresh(adapter->adv_manager);
>  	}
>  
>  	if (changed_mask & MGMT_SETTING_BONDABLE) {
> @@ -4108,6 +4109,14 @@ bool btd_adapter_get_connectable(struct btd_adapter *adapter)
>  	return false;
>  }
>  
> +bool btd_adapter_get_discoverable(struct btd_adapter *adapter)
> +{
> +	if (adapter->current_settings & MGMT_SETTING_DISCOVERABLE)
> +		return true;
> +
> +	return false;
> +}
> +
>  struct btd_gatt_database *btd_adapter_get_database(struct btd_adapter *adapter)
>  {
>  	if (!adapter)
> diff --git a/src/adapter.h b/src/adapter.h
> index a85327cd1..e619a5be9 100644
> --- a/src/adapter.h
> +++ b/src/adapter.h
> @@ -74,6 +74,7 @@ void adapter_foreach(adapter_cb func, gpointer user_data);
>  bool btd_adapter_get_pairable(struct btd_adapter *adapter);
>  bool btd_adapter_get_powered(struct btd_adapter *adapter);
>  bool btd_adapter_get_connectable(struct btd_adapter *adapter);
> +bool btd_adapter_get_discoverable(struct btd_adapter *adapter);
>  
>  struct btd_gatt_database *btd_adapter_get_database(struct btd_adapter *adapter);
>  
> diff --git a/src/advertising.c b/src/advertising.c
> index 94a8c4050..637445189 100644
> --- a/src/advertising.c
> +++ b/src/advertising.c
> @@ -622,8 +622,12 @@ static int refresh_adv(struct btd_adv_client *client, mgmt_request_func_t func)
>  
>  	DBG("Refreshing advertisement: %s", client->path);
>  
> -	if (client->type == AD_TYPE_PERIPHERAL)
> -		flags = MGMT_ADV_FLAG_CONNECTABLE | MGMT_ADV_FLAG_DISCOV;
> +	if (client->type == AD_TYPE_PERIPHERAL) {
> +		flags = MGMT_ADV_FLAG_CONNECTABLE;
> +
> +		if (btd_adapter_get_discoverable(client->manager->adapter))
> +			flags |= MGMT_ADV_FLAG_DISCOV;
> +	}
>  
>  	flags |= client->flags;
>  
> @@ -1104,3 +1108,14 @@ void btd_adv_manager_destroy(struct btd_adv_manager *manager)
>  
>  	manager_destroy(manager);
>  }
> +void btd_adv_manager_refresh(struct btd_adv_manager *manager)
> +{
> +	const struct queue_entry *entry;
> +
> +	for (entry = queue_get_entries(manager->clients); entry;
> +							entry = entry->next) {
> +		struct btd_adv_client *client = entry->data;
> +
> +		refresh_adv(client, NULL);
> +	}
> +}

I think that it is more safe to use queue_foreach() instead of for loop.


Regards,
Eramoto

> diff --git a/src/advertising.h b/src/advertising.h
> index b783cf075..d98c4b864 100644
> --- a/src/advertising.h
> +++ b/src/advertising.h
> @@ -22,3 +22,4 @@ struct btd_adv_manager;
>  
>  struct btd_adv_manager *btd_adv_manager_new(struct btd_adapter *adapter);
>  void btd_adv_manager_destroy(struct btd_adv_manager *manager);
> +void btd_adv_manager_refresh(struct btd_adv_manager *manager);
> 


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] advertising: Configure discoverable flag based on adapter settings
  2018-02-07  1:24 ` ERAMOTO Masaya
@ 2018-02-07  8:46   ` Szymon Janc
  2018-02-07  9:14     ` Luiz Augusto von Dentz
  0 siblings, 1 reply; 5+ messages in thread
From: Szymon Janc @ 2018-02-07  8:46 UTC (permalink / raw)
  To: ERAMOTO Masaya; +Cc: linux-bluetooth

Hi,

On Wednesday, 7 February 2018 02:24:13 CET ERAMOTO Masaya wrote:
> Hi Szymon,
> 
> On 02/06/2018 07:43 PM, Szymon Janc wrote:
> > Until now advertising as peripheral was always setting General
> > Discoverable
> > flag set. With this patch this is set based on adapter discoverable
> > setting. ---
> > 
> >  src/adapter.c     |  9 +++++++++
> >  src/adapter.h     |  1 +
> >  src/advertising.c | 19 +++++++++++++++++--
> >  src/advertising.h |  1 +
> >  4 files changed, 28 insertions(+), 2 deletions(-)
> > 
> > diff --git a/src/adapter.c b/src/adapter.c
> > index 50cf46b11..5070c0f09 100644
> > --- a/src/adapter.c
> > +++ b/src/adapter.c
> > @@ -543,6 +543,7 @@ static void settings_changed(struct btd_adapter
> > *adapter, uint32_t settings)> 
> >  		g_dbus_emit_property_changed(dbus_conn, adapter->path,
> >  		
> >  					ADAPTER_INTERFACE, "Discoverable");
> >  		
> >  		store_adapter_info(adapter);
> > 
> > +		btd_adv_manager_refresh(adapter->adv_manager);
> > 
> >  	}
> >  	
> >  	if (changed_mask & MGMT_SETTING_BONDABLE) {
> > 
> > @@ -4108,6 +4109,14 @@ bool btd_adapter_get_connectable(struct btd_adapter
> > *adapter)> 
> >  	return false;
> >  
> >  }
> > 
> > +bool btd_adapter_get_discoverable(struct btd_adapter *adapter)
> > +{
> > +	if (adapter->current_settings & MGMT_SETTING_DISCOVERABLE)
> > +		return true;
> > +
> > +	return false;
> > +}
> > +
> > 
> >  struct btd_gatt_database *btd_adapter_get_database(struct btd_adapter
> >  *adapter) {
> >  
> >  	if (!adapter)
> > 
> > diff --git a/src/adapter.h b/src/adapter.h
> > index a85327cd1..e619a5be9 100644
> > --- a/src/adapter.h
> > +++ b/src/adapter.h
> > @@ -74,6 +74,7 @@ void adapter_foreach(adapter_cb func, gpointer
> > user_data);> 
> >  bool btd_adapter_get_pairable(struct btd_adapter *adapter);
> >  bool btd_adapter_get_powered(struct btd_adapter *adapter);
> >  bool btd_adapter_get_connectable(struct btd_adapter *adapter);
> > 
> > +bool btd_adapter_get_discoverable(struct btd_adapter *adapter);
> > 
> >  struct btd_gatt_database *btd_adapter_get_database(struct btd_adapter
> >  *adapter);> 
> > diff --git a/src/advertising.c b/src/advertising.c
> > index 94a8c4050..637445189 100644
> > --- a/src/advertising.c
> > +++ b/src/advertising.c
> > @@ -622,8 +622,12 @@ static int refresh_adv(struct btd_adv_client *client,
> > mgmt_request_func_t func)> 
> >  	DBG("Refreshing advertisement: %s", client->path);
> > 
> > -	if (client->type == AD_TYPE_PERIPHERAL)
> > -		flags = MGMT_ADV_FLAG_CONNECTABLE | MGMT_ADV_FLAG_DISCOV;
> > +	if (client->type == AD_TYPE_PERIPHERAL) {
> > +		flags = MGMT_ADV_FLAG_CONNECTABLE;
> > +
> > +		if (btd_adapter_get_discoverable(client->manager->adapter))
> > +			flags |= MGMT_ADV_FLAG_DISCOV;
> > +	}
> > 
> >  	flags |= client->flags;
> > 
> > @@ -1104,3 +1108,14 @@ void btd_adv_manager_destroy(struct btd_adv_manager
> > *manager)> 
> >  	manager_destroy(manager);
> >  
> >  }
> > 
> > +void btd_adv_manager_refresh(struct btd_adv_manager *manager)
> > +{
> > +	const struct queue_entry *entry;
> > +
> > +	for (entry = queue_get_entries(manager->clients); entry;
> > +							entry = entry->next) {
> > +		struct btd_adv_client *client = entry->data;
> > +
> > +		refresh_adv(client, NULL);
> > +	}
> > +}
> 
> I think that it is more safe to use queue_foreach() instead of for loop.

Why? refresh_adv() doesn't modify queue.

-- 
pozdrawiam
Szymon Janc



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] advertising: Configure discoverable flag based on adapter settings
  2018-02-07  8:46   ` Szymon Janc
@ 2018-02-07  9:14     ` Luiz Augusto von Dentz
  2018-02-07  9:16       ` Szymon Janc
  0 siblings, 1 reply; 5+ messages in thread
From: Luiz Augusto von Dentz @ 2018-02-07  9:14 UTC (permalink / raw)
  To: Szymon Janc; +Cc: ERAMOTO Masaya, linux-bluetooth@vger.kernel.org

Hi Szymon, Eramoto,

On Wed, Feb 7, 2018 at 6:46 AM, Szymon Janc <szymon.janc@codecoup.pl> wrote:
> Hi,
>
> On Wednesday, 7 February 2018 02:24:13 CET ERAMOTO Masaya wrote:
>> Hi Szymon,
>>
>> On 02/06/2018 07:43 PM, Szymon Janc wrote:
>> > Until now advertising as peripheral was always setting General
>> > Discoverable
>> > flag set. With this patch this is set based on adapter discoverable
>> > setting. ---
>> >
>> >  src/adapter.c     |  9 +++++++++
>> >  src/adapter.h     |  1 +
>> >  src/advertising.c | 19 +++++++++++++++++--
>> >  src/advertising.h |  1 +
>> >  4 files changed, 28 insertions(+), 2 deletions(-)
>> >
>> > diff --git a/src/adapter.c b/src/adapter.c
>> > index 50cf46b11..5070c0f09 100644
>> > --- a/src/adapter.c
>> > +++ b/src/adapter.c
>> > @@ -543,6 +543,7 @@ static void settings_changed(struct btd_adapter
>> > *adapter, uint32_t settings)>
>> >             g_dbus_emit_property_changed(dbus_conn, adapter->path,
>> >
>> >                                     ADAPTER_INTERFACE, "Discoverable");
>> >
>> >             store_adapter_info(adapter);
>> >
>> > +           btd_adv_manager_refresh(adapter->adv_manager);
>> >
>> >     }
>> >
>> >     if (changed_mask & MGMT_SETTING_BONDABLE) {
>> >
>> > @@ -4108,6 +4109,14 @@ bool btd_adapter_get_connectable(struct btd_adapter
>> > *adapter)>
>> >     return false;
>> >
>> >  }
>> >
>> > +bool btd_adapter_get_discoverable(struct btd_adapter *adapter)
>> > +{
>> > +   if (adapter->current_settings & MGMT_SETTING_DISCOVERABLE)
>> > +           return true;
>> > +
>> > +   return false;
>> > +}
>> > +
>> >
>> >  struct btd_gatt_database *btd_adapter_get_database(struct btd_adapter
>> >  *adapter) {
>> >
>> >     if (!adapter)
>> >
>> > diff --git a/src/adapter.h b/src/adapter.h
>> > index a85327cd1..e619a5be9 100644
>> > --- a/src/adapter.h
>> > +++ b/src/adapter.h
>> > @@ -74,6 +74,7 @@ void adapter_foreach(adapter_cb func, gpointer
>> > user_data);>
>> >  bool btd_adapter_get_pairable(struct btd_adapter *adapter);
>> >  bool btd_adapter_get_powered(struct btd_adapter *adapter);
>> >  bool btd_adapter_get_connectable(struct btd_adapter *adapter);
>> >
>> > +bool btd_adapter_get_discoverable(struct btd_adapter *adapter);
>> >
>> >  struct btd_gatt_database *btd_adapter_get_database(struct btd_adapter
>> >  *adapter);>
>> > diff --git a/src/advertising.c b/src/advertising.c
>> > index 94a8c4050..637445189 100644
>> > --- a/src/advertising.c
>> > +++ b/src/advertising.c
>> > @@ -622,8 +622,12 @@ static int refresh_adv(struct btd_adv_client *client,
>> > mgmt_request_func_t func)>
>> >     DBG("Refreshing advertisement: %s", client->path);
>> >
>> > -   if (client->type == AD_TYPE_PERIPHERAL)
>> > -           flags = MGMT_ADV_FLAG_CONNECTABLE | MGMT_ADV_FLAG_DISCOV;
>> > +   if (client->type == AD_TYPE_PERIPHERAL) {
>> > +           flags = MGMT_ADV_FLAG_CONNECTABLE;
>> > +
>> > +           if (btd_adapter_get_discoverable(client->manager->adapter))
>> > +                   flags |= MGMT_ADV_FLAG_DISCOV;
>> > +   }
>> >
>> >     flags |= client->flags;
>> >
>> > @@ -1104,3 +1108,14 @@ void btd_adv_manager_destroy(struct btd_adv_manager
>> > *manager)>
>> >     manager_destroy(manager);
>> >
>> >  }
>> >
>> > +void btd_adv_manager_refresh(struct btd_adv_manager *manager)
>> > +{
>> > +   const struct queue_entry *entry;
>> > +
>> > +   for (entry = queue_get_entries(manager->clients); entry;
>> > +                                                   entry = entry->next) {
>> > +           struct btd_adv_client *client = entry->data;
>> > +
>> > +           refresh_adv(client, NULL);
>> > +   }
>> > +}
>>
>> I think that it is more safe to use queue_foreach() instead of for loop.
>
> Why? refresh_adv() doesn't modify queue.

In this specific case it is more a matter of taste but I lean toward
using queue_foreach as much as possible as code may change.

> --
> pozdrawiam
> Szymon Janc
>
>
> --
> 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



-- 
Luiz Augusto von Dentz

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] advertising: Configure discoverable flag based on adapter settings
  2018-02-07  9:14     ` Luiz Augusto von Dentz
@ 2018-02-07  9:16       ` Szymon Janc
  0 siblings, 0 replies; 5+ messages in thread
From: Szymon Janc @ 2018-02-07  9:16 UTC (permalink / raw)
  To: Luiz Augusto von Dentz; +Cc: ERAMOTO Masaya, linux-bluetooth@vger.kernel.org

Hi Luiz,

On Wednesday, 7 February 2018 10:14:33 CET Luiz Augusto von Dentz wrote:
> Hi Szymon, Eramoto,
> 
> On Wed, Feb 7, 2018 at 6:46 AM, Szymon Janc <szymon.janc@codecoup.pl> wrote:
> > Hi,
> > 
> > On Wednesday, 7 February 2018 02:24:13 CET ERAMOTO Masaya wrote:
> >> Hi Szymon,
> >> 
> >> On 02/06/2018 07:43 PM, Szymon Janc wrote:
> >> > Until now advertising as peripheral was always setting General
> >> > Discoverable
> >> > flag set. With this patch this is set based on adapter discoverable
> >> > setting. ---
> >> > 
> >> >  src/adapter.c     |  9 +++++++++
> >> >  src/adapter.h     |  1 +
> >> >  src/advertising.c | 19 +++++++++++++++++--
> >> >  src/advertising.h |  1 +
> >> >  4 files changed, 28 insertions(+), 2 deletions(-)
> >> > 
> >> > diff --git a/src/adapter.c b/src/adapter.c
> >> > index 50cf46b11..5070c0f09 100644
> >> > --- a/src/adapter.c
> >> > +++ b/src/adapter.c
> >> > @@ -543,6 +543,7 @@ static void settings_changed(struct btd_adapter
> >> > *adapter, uint32_t settings)>
> >> > 
> >> >             g_dbus_emit_property_changed(dbus_conn, adapter->path,
> >> >             
> >> >                                     ADAPTER_INTERFACE, "Discoverable");
> >> >             
> >> >             store_adapter_info(adapter);
> >> > 
> >> > +           btd_adv_manager_refresh(adapter->adv_manager);
> >> > 
> >> >     }
> >> >     
> >> >     if (changed_mask & MGMT_SETTING_BONDABLE) {
> >> > 
> >> > @@ -4108,6 +4109,14 @@ bool btd_adapter_get_connectable(struct
> >> > btd_adapter
> >> > *adapter)>
> >> > 
> >> >     return false;
> >> >  
> >> >  }
> >> > 
> >> > +bool btd_adapter_get_discoverable(struct btd_adapter *adapter)
> >> > +{
> >> > +   if (adapter->current_settings & MGMT_SETTING_DISCOVERABLE)
> >> > +           return true;
> >> > +
> >> > +   return false;
> >> > +}
> >> > +
> >> > 
> >> >  struct btd_gatt_database *btd_adapter_get_database(struct btd_adapter
> >> >  *adapter) {
> >> >  
> >> >     if (!adapter)
> >> > 
> >> > diff --git a/src/adapter.h b/src/adapter.h
> >> > index a85327cd1..e619a5be9 100644
> >> > --- a/src/adapter.h
> >> > +++ b/src/adapter.h
> >> > @@ -74,6 +74,7 @@ void adapter_foreach(adapter_cb func, gpointer
> >> > user_data);>
> >> > 
> >> >  bool btd_adapter_get_pairable(struct btd_adapter *adapter);
> >> >  bool btd_adapter_get_powered(struct btd_adapter *adapter);
> >> >  bool btd_adapter_get_connectable(struct btd_adapter *adapter);
> >> > 
> >> > +bool btd_adapter_get_discoverable(struct btd_adapter *adapter);
> >> > 
> >> >  struct btd_gatt_database *btd_adapter_get_database(struct btd_adapter
> >> >  *adapter);>
> >> > 
> >> > diff --git a/src/advertising.c b/src/advertising.c
> >> > index 94a8c4050..637445189 100644
> >> > --- a/src/advertising.c
> >> > +++ b/src/advertising.c
> >> > @@ -622,8 +622,12 @@ static int refresh_adv(struct btd_adv_client
> >> > *client,
> >> > mgmt_request_func_t func)>
> >> > 
> >> >     DBG("Refreshing advertisement: %s", client->path);
> >> > 
> >> > -   if (client->type == AD_TYPE_PERIPHERAL)
> >> > -           flags = MGMT_ADV_FLAG_CONNECTABLE | MGMT_ADV_FLAG_DISCOV;
> >> > +   if (client->type == AD_TYPE_PERIPHERAL) {
> >> > +           flags = MGMT_ADV_FLAG_CONNECTABLE;
> >> > +
> >> > +           if (btd_adapter_get_discoverable(client->manager->adapter))
> >> > +                   flags |= MGMT_ADV_FLAG_DISCOV;
> >> > +   }
> >> > 
> >> >     flags |= client->flags;
> >> > 
> >> > @@ -1104,3 +1108,14 @@ void btd_adv_manager_destroy(struct
> >> > btd_adv_manager
> >> > *manager)>
> >> > 
> >> >     manager_destroy(manager);
> >> >  
> >> >  }
> >> > 
> >> > +void btd_adv_manager_refresh(struct btd_adv_manager *manager)
> >> > +{
> >> > +   const struct queue_entry *entry;
> >> > +
> >> > +   for (entry = queue_get_entries(manager->clients); entry;
> >> > +                                                   entry =
> >> > entry->next) {
> >> > +           struct btd_adv_client *client = entry->data;
> >> > +
> >> > +           refresh_adv(client, NULL);
> >> > +   }
> >> > +}
> >> 
> >> I think that it is more safe to use queue_foreach() instead of for loop.
> > 
> > Why? refresh_adv() doesn't modify queue.
> 
> In this specific case it is more a matter of taste but I lean toward
> using queue_foreach as much as possible as code may change.

Fair enough. I'll send V2.

-- 
pozdrawiam
Szymon Janc



^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2018-02-07  9:16 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-02-06 10:43 [PATCH] advertising: Configure discoverable flag based on adapter settings Szymon Janc
2018-02-07  1:24 ` ERAMOTO Masaya
2018-02-07  8:46   ` Szymon Janc
2018-02-07  9:14     ` Luiz Augusto von Dentz
2018-02-07  9:16       ` Szymon Janc

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox