Hi Luiz, On Mon, Mar 4, 2013 at 9:58 AM, Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx> wrote: > + if (g_pattern_match_simple(blacklist[i], profile->name) == 0) > + return -EPROTONOSUPPORT; g_pattern_match_simple() returns gboolean so IMHO it is more readable if you use "== FALSE" or simply "if (!g_pattern_match_simple(...))" above. > + > + if (profile->local_uuid != NULL && strcasecmp(blacklist[i], > + profile->local_uuid) == 0) > + return -EPROTONOSUPPORT; > + > + if (profile->remote_uuids == NULL) > + continue; > + > + if (strcasecmp(blacklist[i], profile->remote_uuids[0]) == 0) > + return -EPROTONOSUPPORT; > + } > + > +done: > profiles = g_slist_append(profiles, profile); > return 0; > } 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
