ipn/ipnlocal: use a switch statement instead of an else-if chain

For clarity.

Signed-off-by: Josh Bleecher Snyder <josh@tailscale.com>
This commit is contained in:
Josh Bleecher Snyder 2021-09-15 12:49:09 -07:00 committed by Josh Bleecher Snyder
parent 7421ba91ec
commit 0f4c0e558b

View File

@ -2357,22 +2357,22 @@ func (b *LocalBackend) nextState() ipn.State {
// Auth was interrupted or waiting for URL visit,
// so it won't proceed without human help.
return ipn.NeedsLogin
} else if state == ipn.Stopped {
}
switch state {
case ipn.Stopped:
// If we were already in the Stopped state, then
// we can assume auth is in good shape (or we would
// have been in NeedsLogin), so transition to Starting
// right away.
return ipn.Starting
} else if state == ipn.NoState {
case ipn.NoState:
// Our first time connecting to control, and we
// don't know if we'll NeedsLogin or not yet.
// UIs should print "Loading..." in this state.
return ipn.NoState
} else if state == ipn.Starting ||
state == ipn.Running ||
state == ipn.NeedsLogin {
case ipn.Starting, ipn.Running, ipn.NeedsLogin:
return state
} else {
default:
b.logf("unexpected no-netmap state transition for %v", state)
return state
}