Hi Marcin, On Wed, Feb 12, 2014, Marcin Kraglak wrote: > + sk = g_io_channel_unix_get_fd(io); > + > + > + if (client_data->data_len != read(sk, buf, client_data->data_len)) { Coding style with the two consecutive empty lines above. Also consider a separate ssize_t ret variable before the length comparison. Johan -- 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