changeset 29700:6eb9c8291a78

debugextensions: simply keep testedwith variable as a list There should be no need to distinguish [] and None.
author Yuya Nishihara <yuya@tcha.org>
date Sun, 10 Jul 2016 21:59:43 +0900
parents 6943dacc3b4c
children 47e48931263e
files mercurial/commands.py
diffstat 1 files changed, 2 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/commands.py	Thu Jul 28 16:27:35 2016 -0400
+++ b/mercurial/commands.py	Sun Jul 10 21:59:43 2016 +0900
@@ -2401,9 +2401,7 @@
     fm = ui.formatter('debugextensions', opts)
     for extname, extmod in sorted(exts, key=operator.itemgetter(0)):
         extsource = extmod.__file__
-        exttestedwith = getattr(extmod, 'testedwith', None)
-        if exttestedwith is not None:
-            exttestedwith = exttestedwith.split()
+        exttestedwith = getattr(extmod, 'testedwith', '').split()
         extbuglink = getattr(extmod, 'buglink', None)
 
         fm.startitem()
@@ -2426,7 +2424,7 @@
                  _('  location: %s\n'), extsource or "")
 
         fm.condwrite(ui.verbose and exttestedwith, 'testedwith',
-                 _('  tested with: %s\n'), ' '.join(exttestedwith or []))
+                 _('  tested with: %s\n'), ' '.join(exttestedwith))
 
         fm.condwrite(ui.verbose and extbuglink, 'buglink',
                  _('  bug reporting: %s\n'), extbuglink or "")