Hi Bharat,
On Thu, Feb 12, 2015, Bharat Panda wrote:
> +static char *ecode_to_string(uint8_t ecode)
This should be static const char *
> +{
> + char *err = NULL;
> +
> + switch (ecode) {
> + case BT_ATT_ERROR_INVALID_HANDLE:
> + err = "ERROR: Invalid Handle";
> + break;
First of all, please follow the coding style: the case statement should
start at the same indentation as the switch. Secondly, you can make this
all more compact by removing the err variable:
case BT_ATT_ERROR_INVALID_HANDLE:
return "Invalid Handle";
case ...:
return "...";
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
