remove unnecessary brackets

scripts/boobank-munin

@@ -149,7 +149,7 @@ class BoobankMuninPlugin(object):
                     except KeyError:
                         pass
             else:
-                accounts = reversed([a for b, a in self.weboob.do('iter_accounts')])
+                accounts = reversed(a for b, a in self.weboob.do('iter_accounts'))
 
             first = True
             for account in accounts:

tools/make_man.py

@@ -75,7 +75,7 @@ class ManpageHelpFormatter(optparse.HelpFormatter):
     def format_option_strings(self, option):
         opts = optparse.HelpFormatter.format_option_strings(self, option).split(", ")
 
-        return ".TP\n"+", ".join(["\\fB%s\\fR" % opt for opt in opts])
+        return ".TP\n"+", ".join("\\fB%s\\fR" % opt for opt in opts)
 
 
 def main():

weboob/applications/havesex/havesex.py

@@ -40,7 +40,7 @@ class ProfileFormatter(IFormatter):
                 result += self.print_node(sub, level+1)
         else:
             if isinstance(node.value, (tuple,list)):
-                value = ','.join([unicode(v) for v in node.value])
+                value = ','.join(unicode(v) for v in node.value)
             else:
                 value = node.value
             result += u'\t' * level + u'%-20s %s\n' % (node.label + ':', value)

weboob/applications/qhavesex/contacts.py

@@ -266,10 +266,10 @@ class ContactProfile(QWidget):
             for sub in node.value:
                 self.process_node(sub, value)
         elif isinstance(node.value, list):
-            value = QLabel('<br />'.join([unicode(s) for s in node.value]))
+            value = QLabel('<br />'.join(unicode(s) for s in node.value))
             value.setWordWrap(True)
         elif isinstance(node.value, tuple):
-            value = QLabel(', '.join([unicode(s) for s in node.value]))
+            value = QLabel(', '.join(unicode(s) for s in node.value))
             value.setWordWrap(True)
         elif isinstance(node.value, (basestring,int,long,float)):
             value = QLabel(unicode(node.value))

weboob/applications/webcontentedit/webcontentedit.py

@@ -88,7 +88,7 @@ class WebContentEdit(ReplApplication):
             print 'No changes. Abort.'
             return
 
-        print 'Contents changed:\n%s' % ('\n'.join([' * %s' % content.id for content in contents]))
+        print 'Contents changed:\n%s' % ('\n'.join(' * %s' % content.id for content in contents))
 
         message = self.ask('Enter a commit message', default='')
 

weboob/backends/aum/pages/profile.py

@@ -433,7 +433,7 @@ class ProfilePage(PageBase):
             for key, value in d.items():
                 key = '%s:' % key
                 if isinstance(value, list):
-                    body += u'\t\t%-15s %s\n' % (key, u', '.join([unicode(s) for s in value]))
+                    body += u'\t\t%-15s %s\n' % (key, u', '.join(unicode(s) for s in value))
                 elif isinstance(value, float):
                     body += u'\t\t%-15s %.2f\n' % (key, value)
                 else:

weboob/tools/application/formatters/csv.py

@@ -36,5 +36,5 @@ class CSVFormatter(IFormatter):
         if self.count == 0:
             result += self.field_separator.join(item.iterkeys()) + '\n'
         self.count += 1
-        result += self.field_separator.join([unicode(v) for v in item.itervalues()])
+        result += self.field_separator.join(unicode(v) for v in item.itervalues())
         return result

weboob/tools/application/qt/backendcfg.py

@@ -287,7 +287,7 @@ class BackendCfg(QDialog):
                                       backend.license,
                                       (unicode(self.tr('<b>Website</b>: %s<br />')) % backend.website) if backend.website else '',
                                       backend.description,
-                                      ', '.join(sorted([cap.__name__.replace('ICap', '') for cap in backend.iter_caps()]))))
+                                      ', '.join(sorted(cap.__name__.replace('ICap', '') for cap in backend.iter_caps()))))
 
         if backend.has_caps(ICapAccount) and self.ui.nameEdit.isEnabled() and backend.klass.ACCOUNT_REGISTER_PROPERTIES is not None:
             self.ui.registerButton.show()

weboob/tools/application/repl.py

@@ -353,7 +353,7 @@ class ReplApplication(Cmd, BaseApplication):
             raise NotEnoughArguments('Command needs %d arguments' % req_n)
 
         if len(args) < nb:
-            args += tuple([None for i in xrange(nb - len(args))])
+            args += tuple(None for i in xrange(nb - len(args)))
         return args
 
     def postcmd(self, stop, line):

weboob/tools/backend.py

@@ -106,7 +106,7 @@ class BaseBackend(object):
         self.lock = RLock()
 
         # Private fields (which start with '_')
-        self._private_config = dict([(key, value) for key, value in config.iteritems() if key.startswith('_')])
+        self._private_config = dict((key, value) for key, value in config.iteritems() if key.startswith('_'))
 
         # Configuration of backend
         self.config = {}

weboob/tools/value.py

@@ -52,7 +52,8 @@ class Value(object):
         if self.regexp is not None and not re.match(self.regexp, unicode(v)):
             raise ValueError('Value "%s" does not match regexp "%s"' % (v, self.regexp))
         if self.choices is not None and not v in self.choices.iterkeys():
-            raise ValueError('Value "%s" is not in list: %s' % (v, ', '.join([unicode(s) for s in self.choices.iterkeys()])))
+            raise ValueError('Value "%s" is not in list: %s' % (
+                v, ', '.join(unicode(s) for s in self.choices.iterkeys())))
 
     def set_value(self, v):
         self.check_valid(v)