On Wed, 2015-05-06 at 14:35 +0300, Jukka Rissanen wrote:
> > + case G_SUPPLICANT_STATE_SCANNING:
> > + if (wifi->connected) {
> > + wifi->connected = false;
> > + start_autoscan(device);
> > + } else
> > + wifi->connected = false;
> > + break;
This needs to be explicit or else no-one (me!) will not realize this
later on and just contract the wifi->connected cases like Jukka
suggested. So perhaps do an 'old_state = wifi->connected;' and then set
connected to false so that there are some warning signs around the code.
Cheers,
Patrik
_______________________________________________
connman mailing list
[email protected]
https://lists.connman.net/mailman/listinfo/connman
