Hi Anderson,
2011/11/9 Anderson Lizardo <anderson.lizardo@xxxxxxxxxxxxx>:
> Hi Santiago,
>
> On Wed, Nov 9, 2011 at 6:52 AM, Santiago Carot-Nemesio
> <sancane@xxxxxxxxx> wrote:
>> static void recv_measurement(struct thermometer *t, struct measurement *msmt)
>> {
>> - GSList *l;
>> + GSList *l, *ll;
>>
>> - if (g_strcmp0(msmt->msmnt, "Intermediate") == 0) {
>> - DBG("Notification of intermediate measurement not implemented");
>> - return;
>> - }
>> + if (g_strcmp0(msmt->msmnt, "Intermediate") == 0)
>> + ll = t->iwatchers;
>> + else
>> + ll = t->fwatchers;
>>
>> - for (l = t->fwatchers; l; l = l->next)
>> + for (l = ll; l; l = l->next)
>> update_watcher(l->data, msmt);
>> }
>
> What about using g_slist_foreach() above? Also "ll" seems too generic,
> what about "wlist".
Ok, I'll prepare a new set of patches, but I would rather wait to get
comments and advices to the question regarding to how we should manage
GATT attribute indications.
Thank you very much for your comments.
Santiago
>
> Regards,
> --
> Anderson Lizardo
> Instituto Nokia de Tecnologia - INdT
> Manaus - Brazil
>
--
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
