more whitespace cleanup and some other style nits

hgext/convert/__init__.py

@@ -124,7 +124,7 b' def convert(ui, src, dest=None, revmapfi'
         --config convert.cvsps=builtin
     and has a few more configurable options:
         --config convert.cvsps.cache=True     (boolean)
-            Set to False to disable remote log caching, for testing and 
+            Set to False to disable remote log caching, for testing and
             debugging purposes.
         --config convert.cvsps.fuzz=60        (integer)
             Specify the maximum time (in seconds) that is allowed

hgext/rebase.py

@@ -222,7 +222,7 b' def rebasenode(repo, rev, target, state,'
                 repo.dirstate.copy(v, k)
                 if v in m2 and v not in m1:
                     repo.dirstate.remove(v)
-                
+
     newrev = concludenode(repo, rev, p1, p2, state, collapse,
                           extrafn=extrafn)
 

mercurial/config.py

@@ -20,7 +20,7 b' class sortdict(dict):'
         for k in src:
             self[k] = src[k]
     def items(self):
-        return [(k,self[k]) for k in self._list]
+        return [(k, self[k]) for k in self._list]
     def __delitem__(self, key):
         dict.__delitem__(self, key)
         self._list.remove(key)

mercurial/ui.py

@@ -9,8 +9,8 b' from i18n import _'
 import errno, getpass, os, re, socket, sys, tempfile
 import config, traceback, util, error
 
-_booleans = {'1':True, 'yes':True, 'true':True, 'on':True,
-             '0':False, 'no':False, 'false':False, 'off':False}
+_booleans = {'1': True, 'yes': True, 'true': True, 'on': True,
+             '0': False, 'no': False, 'false': False, 'off': False}
 
 class ui(object):
     def __init__(self, src=None):
@@ -34,6 +34,7 b' class ui(object):'
             # we always trust global config files
             for f in util.rcpath():
                 self.readconfig(f, trust=True)
+
     def copy(self):
         return self.__class__(self)
 
@@ -57,7 +58,7 b' class ui(object):'
         return False
 
     def readconfig(self, filename, root=None, trust=False,
-                   sections = None):
+                   sections=None):
         try:
             fp = open(filename)
         except IOError:
@@ -125,7 +126,7 b' class ui(object):'
             uvalue = self._ucfg.get(section, name)
             if uvalue is not None and uvalue != value:
                 self.debug(_("ignoring untrusted configuration option "
-                            "%s.%s = %s\n") % (section, name, uvalue))
+                             "%s.%s = %s\n") % (section, name, uvalue))
         return value
 
     def configbool(self, section, name, default=False, untrusted=False):
@@ -153,7 +154,7 b' class ui(object):'
     def configitems(self, section, untrusted=False):
         items = self._data(untrusted).items(section)
         if self.debugflag and not untrusted and self._reportuntrusted:
-            for k,v in self._ucfg.items(section):
+            for k, v in self._ucfg.items(section):
                 if self._tcfg.get(section, k) != v:
                     self.debug(_("ignoring untrusted configuration option "
                                 "%s.%s = %s\n") % (section, k, v))

tests/test-convert.out

@@ -109,7 +109,7 b' convert a foreign SCM repository to a Me'
         --config convert.cvsps=builtin
     and has a few more configurable options:
         --config convert.cvsps.cache=True     (boolean)
-            Set to False to disable remote log caching, for testing and 
+            Set to False to disable remote log caching, for testing and
             debugging purposes.
         --config convert.cvsps.fuzz=60        (integer)
             Specify the maximum time (in seconds) that is allowed