Hi Pekka,

On Mon, Jan 24, 2011 at 07:55:36PM +0200, pekka.pe...@nokia.com wrote:
> From: Pekka Pessi <pekka.pe...@nokia.com>
> 
> Once connect succeeds, allow retrying autoconnect.
> ---
>  src/service.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/src/service.c b/src/service.c
> index c3085ef..3a4260a 100644
> --- a/src/service.c
> +++ b/src/service.c
> @@ -4519,6 +4519,9 @@ static int service_load(struct connman_service *service)
>               service->favorite = g_key_file_get_boolean(keyfile,
>                               service->identifier, "Favorite", NULL);
>  
> +             if (service->favorite == TRUE)
> +                     set_reconnect_state(service, TRUE);
> +
I would prefer to see that being done from your
__connman_service_clear_error() routine, when the service is a favorite one.
Setting reconnect state to TRUE means the service layer know that we currently
managed to connect to it once, which is not the case from service_load.

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/
_______________________________________________
connman mailing list
connman@connman.net
http://lists.connman.net/listinfo/connman

Reply via email to