Excerpts from Simon Schampijer's message of Wed Mar 23 15:56:54 +0100 2011:

[src/jarabe/model/neighborhood.py]
> +        params_needing_reconnect = account.UpdateParameters(
> +            {'server': server,
> +             'account': account_id,
> +             'register': True},
> +            dbus.Array([], 's'), dbus_interface=ACCOUNT)

The dictionary fits on a single line, increasing the amount of code that
fits on the screen and thus making it easier to grok the code when
viewing it on an XO.

> +        params_needing_reconnect = account.UpdateParameters(
> +            {'jid': account_id},
> +            dbus.Array([], 's'), dbus_interface=ACCOUNT)

Similarly, all parameters fit on a single line.


Acked-by: Sascha Silbe <si...@activitycentral.com>


Thanks for the patch!

Sascha

-- 
http://sascha.silbe.org/
http://www.infra-silbe.de/

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Sugar-devel mailing list
Sugar-devel@lists.sugarlabs.org
http://lists.sugarlabs.org/listinfo/sugar-devel

Reply via email to