Hi Marcel, On Wed, Oct 09, 2013 at 09:14:05PM +0200, Marcel Holtmann wrote: > > +static bt_status_t listen(btsock_type_t type, const char *service_name, > > + const uint8_t *uuid, int chan, > > + int *sock, int flags) > > +{ > > + if ((uuid == NULL && chan <= 0) || sock == NULL) { > > we are moving away from uuid == NULL checks. Use !uuid instead. I do not like myself these kind of checks but since this seems to be BlueZ style. Should I change comparison val != NULL and val != 0 to !val ? Like in kernel? Best regards Andrei Emeltchenko -- 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