Hi,

On Mon, 2012-03-26 at 11:40 +0300, Alok Barsode wrote:
>         int i;
>  
>         DBG("service %p", service);
> @@ -248,6 +264,13 @@ static int timeserver_start(struct
> connman_service *service)
>         /* Cache service timeservers */
>         service_ts = connman_service_get_timeservers(service);
>  
> +       network = __connman_service_get_network(service);
> +
> +       i = connman_network_get_index(network); 

Nitpick: I would call this instance of 'i' 'index', 'ifindex' or 'idx'
to distinguish it from a temporary loop variable 'i' found e.g. in
__connman_timeserver_sync().


Cheers,

        Patrik


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

Reply via email to