Re: [PATCH v3] Simplify eir_parse_uuid* functions

Hi Frédéric,

2011/10/20 Frédéric Danis [off-list ref]:

+                       if (eir->name)
+                               g_free(eir->name);

You don't need to check for NULL before passing a pointer to g_free().
IMHO just "g_free(eir->name)" is clearer.

+
+                       eir->name = g_strndup((char *) &eir_data[2],
                                                               field_len - 1);
-                       else
-                               eir->name = g_strdup("");
                       eir->name_complete = eir_data[1] == EIR_NAME_COMPLETE;
                       break;

Regards,
-- 
Anderson Lizardo
Instituto Nokia de Tecnologia - INdT
Manaus - Brazil