Hi Tomasz,

Some nitpicking:

On Tue, Jan 11, 2011 at 03:54:02PM +0200, Tomasz Bursztyka wrote:
> ---
>  test/get-services  |    2 +-
>  test/list-services |    2 +-
>  test/test-manager  |    2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/test/get-services b/test/get-services
> index 29d4ea0..605e278 100755
> --- a/test/get-services
> +++ b/test/get-services
> @@ -43,7 +43,7 @@ for entry in services:
>                                       "Domains", "Domains.Configuration"]:
>                       val = extract_list(properties[key])
>               elif key in ["Favorite", "Immutable", "AutoConnect",
> -                             "SetupRequired", "PassphraseRequired"]:
> +                             "SetupRequired", "PassphraseRequired", "WPS"]:
>                       if properties[key] == dbus.Boolean(1):
>                               val = "true"
>                       else:
> diff --git a/test/list-services b/test/list-services
> index 66dfe5c..f1b89b6 100755
> --- a/test/list-services
> +++ b/test/list-services
> @@ -49,7 +49,7 @@ for path in properties["Services"]:
>                       val = extract_list(properties[key])
>               elif key in ["Favorite", "Immutable", "AutoConnect",
>                                       "LoginRequired", "SetupRequired",
> -                                                     "PassphraseRequired"]:
> +                                             "PassphraseRequired", "WPS"]:
>                       if properties[key] == dbus.Boolean(1):
>                               val = "true"
>                       else:
> diff --git a/test/test-manager b/test/test-manager
> index b5642d0..9f35105 100755
> --- a/test/test-manager
> +++ b/test/test-manager
> @@ -56,7 +56,7 @@ def print_properties(key, value):
>                                       "Available", "Remember", "Default",
>                                       "Favorite", "Immutable", "AutoConnect",
>                                               "LoginRequired", 
> "SetupRequired",
> -                                                     "PassphraseRequired"]:
> +                                                     "PassphraseRequired", 
> "WPS"]:
>
more than 80 chars here.

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