Hi Anderson,
2011/10/20 Anderson Lizardo <anderson.lizardo@xxxxxxxxxxxxx>:
> Hi Santiago,
>
> On Thu, Oct 20, 2011 at 6:46 AM, Santiago Carot-Nemesio
> <sancane@xxxxxxxxx> wrote:
>> +static void final_measurement_cb (guint8 status, const guint8 *pdu,
>> + guint16 len, gpointer user_data)
>
> There should be no space before "(" in function definitions/declarations.
>
>> +{
>> + gchar *msg = user_data;
>> +
>> + if (status != 0) {
>> + error("%s failed", msg);
>
> You don't need to g_free(msg) in case of error?
Yes I do, good catch!, I'll work in a new set of patches.
Thank for your comments.
>
>> + return;
>> + }
>> +
>> + g_free(msg);
>> +}
>> +
>
> 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
