Tim Foster wrote:

> https://cr.opensolaris.org/action/browse/pkg/timf/l10n-cleanup/

setup.py:

  - You don't use StringIO.

client.py:

  - you use "str(e)" a lot, even though this should be covered by the %s
    format, and even though it wasn't always there before.  Here and in
    other files.

  - line 1387: need a space between "%" and "{"

pkg_solver.py:

  - line 1715: not necessarily for RTL languages.

  - line 1757: this probably still needs translation for RTL languages.

progress.py:

  - line 1974: no need for a continuation character

Thanks,
Danek
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to