On 05.04.2012 15:04, Daniel Wagner wrote:
> From: Daniel Wagner <[email protected]>
> 
> ---
>  src/service.c |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/src/service.c b/src/service.c
> index 36140f9..6be617f 100644
> --- a/src/service.c
> +++ b/src/service.c
> @@ -2882,6 +2882,11 @@ static connman_bool_t is_ignore(struct connman_service 
> *service)
>       if (service->state == CONNMAN_SERVICE_STATE_FAILURE)
>               return TRUE;
>  
> +     if (service->network != NULL) {
> +             if (connman_network_get_device(service->network) == NULL)
> +                     return TRUE;
> +     }
> +
>       return FALSE;
>  }
>  

I think, it would make sense to introduce something like

__connman_network_is_shutdown()

(of course a better name would be nice, ideas), instead of this
network->device != NULL magic. What do you think?


cheers,
daniel



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

Reply via email to