[Sugar-devel] [PATCH sugar-base 00/17] style cleanup series

Sascha Silbe sascha-pgp at silbe.org
Sat Oct 16 07:30:13 EDT 2010


This is the sugar-base part of the style cleanups I've wanted to
land at least two releases ago, but kept missing the window.

I have done some basic testing, but naturally cannot test all code
paths I've touched (some even appear to be unused). This is why I'd like
to land this right after we branched off sucrose-0.90: to ensure it gets
enough testing before the next release.

After this series sugar-base is pep8 and pylint clean.

Sascha Silbe (17):
  fix EOL spaces
  maint-helper.py: remove outdated URL
  PEP8 cleanup: fix number of blank lines
  sugar.dispatch.saferef: don't use assert for critical code paths
  PEP8 cleanup: fix spaces around operators and parentheses
  sugar.dispatch.dispatcher: remove fallback for Python 2.3
  PEP8 cleanup: ensure lines are shorter than 80 characters
  PEP8 cleanup: don't use has_key()
  PEP8 cleanup: import only one module per import statement
  style cleanup: prefer ' for strings
  pylint cleanup: replace disable-msg with disable
  pylint: sugar.logger: allow overriding module name
  maint-helper: rename parameter to avoid name clash with built-in
    function
  sugar.dispatch.saferef: ignore incorrect pylint warning
  pylint cleanup: sugar.dispatch.saferef: remove unused variable
  pylint cleanup: Signal._live_receivers: mark receiverkey as unused
  sugar.mime: ignore incorrect pylint warning

 maint-helper.py                  |   40 +++++++++------
 src/sugar/dispatch/__init__.py   |    3 +-
 src/sugar/dispatch/dispatcher.py |   42 ++++++++--------
 src/sugar/dispatch/saferef.py    |  102 ++++++++++++++++++++------------------
 src/sugar/logger.py              |   14 +++---
 src/sugar/mime.py                |    1 +
 6 files changed, 107 insertions(+), 95 deletions(-)



More information about the Sugar-devel mailing list