6 Apr
2023
6 Apr
'23
11:10 p.m.
On Sat, Apr 01, 2023 at 09:30:05AM +0200, Heinrich Schuchardt wrote:
gcc 12 does not understand /* Fall through */. Use the fallthrough macro instead.
Fallthrough at the start of a switch statement makes no sense.
Signed-off-by: Heinrich Schuchardt heinrich.schuchardt@canonical.com Reviewed-by: Ramon Fried rfried.dev@gmail.com Reviewed-by: Simon Glass sjg@chromium.org
[snip]
@@ -1484,11 +1483,11 @@ static int net_check_prereq(enum proto_t protocol) puts("*** ERROR: `serverip' not set\n"); return 1; }
fallthrough;#if defined(CONFIG_CMD_PING) || \ defined(CONFIG_CMD_DNS) || defined(CONFIG_PROT_UDP) common: #endif
This leads to: net/net.c:1486:3: error: fallthrough annotation does not directly precede switch label with clang.
--
Tom