bartoszek/zephyr
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

net: dhcpv6: adjust switch-case in dhcpv6_enter_state

Browse Source 
For code clarity, unified switch-case usage in `dhcpv6_enter_state` to
use `break` instead of `return`.
Typically, a `break` is used in switch-case statements unless an early
return is necessary, in which case `return` is appropriate.

In this scenario, the `break` statement is the more suitable choice.

Signed-off-by: Pisit Sawangvonganan <pisit@ndrsolution.com>
This commit is contained in:
Pisit Sawangvonganan 2024-09-18 15:28:17 +07:00 committed by Henrik Brix Andersen
parent 726ac5cc06
commit 44ba8a5485
1 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
subsys/net/lib/dhcpv6/dhcpv6.c
View File

@ -1417,21 +1417,28 @@ static void dhcpv6_enter_state(struct net_if *iface, enum net_dhcpv6_state state
case NET_DHCPV6_DISABLED:
break;
case NET_DHCPV6_INIT:
return dhcpv6_enter_init(iface);
dhcpv6_enter_init(iface);
break;
case NET_DHCPV6_SOLICITING:
return dhcpv6_enter_soliciting(iface);
dhcpv6_enter_soliciting(iface);
break;
case NET_DHCPV6_REQUESTING:
return dhcpv6_enter_requesting(iface);
dhcpv6_enter_requesting(iface);
break;
case NET_DHCPV6_CONFIRMING:
return dhcpv6_enter_confirming(iface);
dhcpv6_enter_confirming(iface);
break;
case NET_DHCPV6_RENEWING:
return dhcpv6_enter_renewing(iface);
dhcpv6_enter_renewing(iface);
break;
case NET_DHCPV6_REBINDING:
return dhcpv6_enter_rebinding(iface);
dhcpv6_enter_rebinding(iface);
break;
case NET_DHCPV6_INFO_REQUESTING:
break;
case NET_DHCPV6_BOUND:
return dhcpv6_enter_bound(iface);
dhcpv6_enter_bound(iface);
break;
}
}