Best viewed with whitespace changes off as there aren't actually many changes here.
I've just inverted the first two if statements and made them return early instead of having loads of nested if statements.
Then I've removed the call add_seen_bssid on a newly constructed SettingsWireless object as it apparently doesn't do anything and probably never did anything anyway, since that object is never used again.
The only things that need testing for regressions here is connecting to WiFi networks, though I anticipate no regressions.
Best viewed with whitespace changes off as there aren't actually many changes here.
I've just inverted the first two if statements and made them return early instead of having loads of nested if statements.
Then I've removed the call
add_seen_bssid
on a newly constructedSettingsWireless
object as it apparently doesn't do anything and probably never did anything anyway, since that object is never used again.The only things that need testing for regressions here is connecting to WiFi networks, though I anticipate no regressions.