[Sugar-devel] [PATCH] Simple NetworkManager-0.9 port

Frederick Grose fgrose at gmail.com
Mon Oct 24 15:30:59 EDT 2011


On Mon, Oct 24, 2011 at 1:41 PM, Sascha Silbe <
sascha-ml-reply-to-2011-4 at silbe.org> wrote:

> Excerpts from Daniel Drake's message of 2011-09-15 21:36:58 +0200:
>


{...}


> > +    def _secrets_err_cb(self, err):
> > +        logging.debug("No GSM secrets present, or error: %s", err)
>
> In Sugar source, we prefer single quotes over double quotes for string
> literals (unless the literal contains single quotes). An arbitrary
> decision, but for consistency we should use the same style everywhere.
> This applies to same other parts of the patch as well.


http://wiki.sugarlabs.org/go/Development_Team/Code_guidelines#Miscellaneous

There is a human factors reason for preferring single over double quotes:

In a busy field of view, such as code, maximizing the 'information/ink'
ratio (by removing superfluous ink or pixels) makes it easier to 'see' or
grasp what is important in all the eye spans that need to be mentally
integrated.

        --Fred

{...}
> --
> http://sascha.silbe.org/
> http://www.infra-silbe.de/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20111024/52842700/attachment.html>


More information about the Sugar-devel mailing list