[MERGE] Explicitly list aliases in the manpage as commands that can be executed. (#336998)

Ben Finney bignose+hates-spam at benfinney.id.au
Wed Mar 4 03:12:21 GMT 2009


Ian Clatworthy <ian.clatworthy at canonical.com> writes:

> +    for cmd_name in sorted(formatted.keys()):
> +        output = output + formatted[cmd_name]
> 
> .keys() is implicit I think.

Yes.

> You can use output += ... as well.

Even better to use str.join(), fed with a generator expression:

    output = "".join(value for (name, value) in sorted(formatted.items()))

-- 
 \      “He who allows oppression, shares the crime.” —Erasmus Darwin, |
  `\                                     grandfather of Charles Darwin |
_o__)                                                                  |
Ben Finney




More information about the bazaar mailing list