Hi Mikel,
On Tue, Mar 26, 2013, Mikel Astiz wrote:
> +static const char *state2str(btd_service_state_t state)
> +{
> + switch(state) {
> + case BTD_SERVICE_STATE_UNAVAILABLE:
> + return "BTD_SERVICE_STATE_UNAVAILABLE";
> + case BTD_SERVICE_STATE_DISCONNECTED:
> + return "BTD_SERVICE_STATE_DISCONNECTED";
> + case BTD_SERVICE_STATE_CONNECTING:
> + return "BTD_SERVICE_STATE_CONNECTING";
> + case BTD_SERVICE_STATE_CONNECTED:
> + return "BTD_SERVICE_STATE_CONNECTED";
> + case BTD_SERVICE_STATE_DISCONNECTING:
> + return "BTD_SERVICE_STATE_DISCONNECTING";
> + }
Minor thing: Could you just make the strings "unavailable",
"disconnected", etc. The context is already evident from the rest of the
logs (especially the DBG ones) and having something shorter would make
them more readable.
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
