[review] dcbw/mbm-connect-disconnect-cleanup: simplify and clean up mbm plugin connect and disconnect

[Aleksander Morgado]

On Wed, May 11, 2016 at 8:26 PM, Dan Williams <dcbw at redhat.com> wrote:
> Please review the dcbw/mbm-connect-disconnect-cleanup branch.
>
> https://cgit.freedesktop.org/ModemManager/ModemManager/log/?h=dcbw/mbm-connect-disconnect-cleanup
>
> Git commit comments should have a pretty good overview of what's going
> on; these were spawned out of the following bugs that Lubomir found
> doing some automated testing:
>
> https://bugs.freedesktop.org/show_bug.cgi?id=95302
> https://bugs.freedesktop.org/show_bug.cgi?id=95304

LGTM; but for the last one, "broadband-bearer-mbm: connect errors
should trigger a disconnect", shouldn't we do this in the more generic
MMBroadbandBearer or MMBearer instead? Don't we already have something
like that?

-- 
Aleksander
https://aleksander.es