Hi Daniel,

On 04/11/2012 11:19 AM, Daniel Wagner wrote:
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?

Good idea.

Other name options could be:

__connman_network_is_connectable()
__connman_network_is_disconnected()
__connman_network_is_valid()



cheers,
daniel


Cheers,
Jukka

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

Reply via email to