Hi Gustavo,

> >> Attached are simple patches to add notifications that the following
> >> properties changed:
> >>
> >> 0001-notify-manager-property-changed-when-system-goes-off.patch
> >>       if the default profiles go offline, then notify manager property
> >> OfflineMode changed.
> >>
> >> 0002-service-notify-IPv4.Configuration-property-changed.patch
> >>      when IPv4.Configuration changes, notify it
> >
> > both patches have been applied. However I prefer one patch per email.
> > And actually using git send-email makes them nice inline patches instead
> > of attachments.
> >
> >> Some others are still missing, but I need more time to figure out how
> >> to proper notify them. One example is that Manager
> >> Services/Networks/Devices are not being updated when these go away,
> >> this is easy to spot when you turn offline mode and your client will
> >> not get PropertyChanged on manager.
> >
> > Send me patches for it ;)
> 
> let's see when I'll have time to figure out the workflow and do the
> patch. I did a quick look and seems that network_remove() should alert
> the profile, and __connman_profile_remove_network() alerts service,
> and  __connman_service_remove_from_network() should alert manager...
> or service_free() itself should do it?

that is code flow through it. You are on the right path.

Regards

Marcel


_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to