Re: [PATCH V3 7/9] mesh: meshctl: Add get/set identity

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Steve,

On Fri, Dec 15, 2017, sbrown@xxxxxxxxxxxx wrote:
> +	case OP_NODE_IDENTITY_STATUS:
> +		if (len != 4)
> +			return true;
> +		bt_shell_printf("Network index 0x%04x has "
> +				"Node Identity state 0x%02x %s\n",
> +				get_le16(data + 1), data[3],
> +				mesh_status_str(data[0]));
> +
>  	case OP_CONFIG_PROXY_STATUS:

This throws the following compilation error for me:

mesh/config-client.c: In function ‘client_msg_recvd’:
mesh/config-client.c:165:3: error: this statement may fall through [-Werror=implicit-fallthrough=]
   bt_shell_printf("Network index 0x%04x has "
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     "Node Identity state 0x%02x %s\n",
     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     get_le16(data + 1), data[3],
     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     mesh_status_str(data[0]));
     ~~~~~~~~~~~~~~~~~~~~~~~~~
mesh/config-client.c:170:2: note: here
  case OP_CONFIG_RELAY_STATUS:
  ^~~~
cc1: all warnings being treated as errors

The above looks like a valid error, i.e. it seems there's a missing
break or return statement in your patch.

You might want to upgrade to a newer gcc version if you're not getting
these errors. At least version 7.2.1 from Fedora 27 give me these.

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



[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux