diff options
| author | Alexander Couzens <lynxis@fe80.eu> | 2018-06-29 04:30:13 +0200 |
|---|---|---|
| committer | Alexander Couzens <lynxis@fe80.eu> | 2018-07-11 22:03:38 +0200 |
| commit | 60293a7a3656aab8c10f2d1827d1da88ad46d32c (patch) | |
| tree | 21e820844c861085dab08403e5da04f94ce43aa9 /interface.c | |
| parent | 5cf79759a24e9bb2a6a3aef7c83d73efb9bf2df3 (diff) | |
| download | netifd-60293a7a3656aab8c10f2d1827d1da88ad46d32c.tar.gz | |
replace fall throughs in switch/cases where possible with simple code changes
fall throughs are usually error-prone, especially when someone else extend
it.
Signed-off-by: Alexander Couzens <lynxis@fe80.eu>
Acked-by: Hans Dedecker <dedeckeh@gmail.com>
Diffstat (limited to 'interface.c')
| -rw-r--r-- | interface.c | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/interface.c b/interface.c index 400c605..e32f08a 100644 --- a/interface.c +++ b/interface.c @@ -387,26 +387,28 @@ static void interface_main_dev_cb(struct device_user *dep, enum device_event ev) { struct interface *iface; - bool new_state = false; iface = container_of(dep, struct interface, main_dev); switch (ev) { case DEV_EVENT_ADD: - new_state = true; + interface_set_available(iface, true); + break; case DEV_EVENT_REMOVE: - interface_set_available(iface, new_state); - if (!new_state && dep->dev && dep->dev->external) + interface_set_available(iface, false); + if (dep->dev && dep->dev->external) interface_set_main_dev(iface, NULL); break; case DEV_EVENT_UP: - new_state = true; + interface_set_enabled(iface, true); + break; case DEV_EVENT_DOWN: - interface_set_enabled(iface, new_state); + interface_set_enabled(iface, false); break; case DEV_EVENT_LINK_UP: - new_state = true; + interface_set_link_state(iface, true); + break; case DEV_EVENT_LINK_DOWN: - interface_set_link_state(iface, new_state); + interface_set_link_state(iface, false); break; case DEV_EVENT_TOPO_CHANGE: interface_proto_event(iface->proto, PROTO_CMD_RENEW, false); |