]> granicus.if.org Git - python/commitdiff
remove most uses of list(somedict.keys()) in Demo scripts
authorSkip Montanaro <skip@pobox.com>
Mon, 6 Aug 2007 21:07:53 +0000 (21:07 +0000)
committerSkip Montanaro <skip@pobox.com>
Mon, 6 Aug 2007 21:07:53 +0000 (21:07 +0000)
17 files changed:
Demo/cgi/cgi2.py
Demo/classes/Dbm.py
Demo/metaclasses/Enum.py
Demo/newmetaclasses/Enum.py
Demo/pdist/cmdfw.py
Demo/pdist/cmptree.py
Demo/pdist/cvslib.py
Demo/pdist/rrcs.py
Demo/pdist/server.py
Demo/scripts/ftpstats.py
Demo/scripts/markov.py
Demo/scripts/newslist.py
Demo/threads/Coroutine.py
Demo/tix/samples/OptMenu.py
Demo/tkinter/guido/AttrDialog.py
Demo/xml/elem_count.py
Demo/xml/roundtrip.py

index 239fb6b5f95f4d818deec4f079d1164fbedbb53b..1d5822c463f996ce8655a9b671b7fb6c0bb01e90 100755 (executable)
@@ -14,7 +14,7 @@ def main():
         print("<h1>No Form Keys</h1>")
     else:
         print("<h1>Form Keys</h1>")
-        for key in list(form.keys()):
+        for key in form.keys():
             value = form[key].value
             print("<p>", cgi.escape(key), ":", cgi.escape(value))
 
index fd735718760746617a1e5dd0de93a8ed052c672d..ecc117d495379cbfb0c8fea8bdb88c74773711cd 100755 (executable)
@@ -12,7 +12,7 @@ class Dbm:
 
     def __repr__(self):
         s = ''
-        for key in list(self.keys()):
+        for key in self.keys():
             t = repr(key) + ': ' + repr(self[key])
             if s: t = ', ' + t
             s = s + t
@@ -32,7 +32,7 @@ class Dbm:
 
     def keys(self):
         res = []
-        for key in list(self.db.keys()):
+        for key in self.db.keys():
             res.append(eval(key))
         return res
 
index 8d402db97753330285232328ed8891509c562dc3..eb52d79cee2e29711b52fdf2ec7730e6d470a929 100644 (file)
@@ -42,7 +42,7 @@ class EnumMetaClass:
         self.__name__ = name
         self.__bases__ = bases
         self.__dict = {}
-        for key, value in list(dict.items()):
+        for key, value in dict.items():
             self.__dict[key] = EnumInstance(name, key, value)
 
     def __getattr__(self, name):
@@ -80,7 +80,7 @@ class EnumMetaClass:
             s = s + '(' + string.join([x.__name__ for x in self.__bases__], ", ") + ')'
         if self.__dict:
             list = []
-            for key, value in list(self.__dict.items()):
+            for key, value in self.__dict.items():
                 list.append("%s: %s" % (key, int(value)))
             s = "%s: {%s}" % (s, string.join(list, ", "))
         return s
index 0f7f913fb09e46c2674a412831bf690c5707b8f0..99b6fe80415380114a0988529f7532ff321fdefa 100644 (file)
@@ -20,7 +20,7 @@ class EnumMetaclass(type):
     def __init__(cls, name, bases, dict):
         super(EnumMetaclass, cls).__init__(name, bases, dict)
         cls._members = []
-        for attr in list(dict.keys()):
+        for attr in dict.keys():
             if not (attr.startswith('__') and attr.endswith('__')):
                 enumval = EnumInstance(name, attr, dict[attr])
                 setattr(cls, attr, enumval)
index 7bcb4616b78e607ced01132a0ef6dac216a4b1e4..ec854b1bc743a81fa761c50b54a658ef81383ac5 100755 (executable)
@@ -104,9 +104,7 @@ class CommandFrameWork:
             c = c.__bases__[0]
         if docstrings:
             print("where subcommand can be:")
-            names = list(docstrings.keys())
-            names.sort()
-            for name in names:
+            for name in sorted(docstrings.keys()):
                 print(docstrings[name])
         if self.PostUsageMessage:
             print(self.PostUsageMessage)
index d6962e95ad3dd7cbeb54c4540e04de0fc6e226f1..1e9f985e72166da1f2c6ddf67b6cd20662916176 100755 (executable)
@@ -89,7 +89,7 @@ def compare(local, remote, mode):
                 else:
                     print("same mtime but different sum?!?!", end=' ')
                 print()
-    for name in list(lsumdict.keys()):
+    for name in lsumdict.keys():
         if not list(rsumdict.keys()):
             print(repr(name), "only locally", end=' ')
             fl()
index 8ccb6166bd6a1f72c9cb24d8f624960bed2aa032..78e4fbb4c9df4404a77e4ee9df2001417b98b2c7 100755 (executable)
@@ -223,15 +223,12 @@ class CVS:
         f.close()
 
     def getlocalfiles(self):
-        list = list(self.entries.keys())
+        entries_keys = set(self.entries.keys())
         addlist = os.listdir(os.curdir)
         for name in addlist:
-            if name in list:
-                continue
             if not self.ignored(name):
-                list.append(name)
-        list.sort()
-        for file in list:
+                entries_keys.add(name)
+        for file in sorted(entries_keys):
             try:
                 e = self.entries[file]
             except KeyError:
@@ -257,19 +254,17 @@ class CVS:
         print('-'*50)
 
     def keys(self):
-        keys = list(self.entries.keys())
-        keys.sort()
-        return keys
+        return sorted(self.entries.keys())
 
     def values(self):
         def value(key, self=self):
             return self.entries[key]
-        return list(map(value, list(self.keys())))
+        return [value(k) for k in self.keys()]
 
     def items(self):
         def item(key, self=self):
             return (key, self.entries[key])
-        return list(map(item, list(self.keys())))
+        return [item(k) for k in self.keys()]
 
     def cvsexists(self, file):
         file = os.path.join("CVS", file)
index 8d0ef0358da20e5c7ef1c6e7fd3dcf51dc7ad62c..647ecc5c0549e105820f4c74dabc8714d69dbd22 100755 (executable)
@@ -71,11 +71,9 @@ def unlock(x, copts, fn):
     x.unlock(fn)
 
 def info(x, copts, fn):
-    dict = x.info(fn)
-    keys = list(dict.keys())
-    keys.sort()
-    for key in keys:
-        print(key + ':', dict[key])
+    info_dict = x.info(fn)
+    for key in sorted(info_dict.keys()):
+        print(key + ':', info_dict[key])
     print('='*70)
 
 def head(x, copts, fn):
index 98a47411d8facffbaa1dae803d31c7a838824ade..7e779712a9ee083df5cc387b7f9d6ebc9056bb7e 100755 (executable)
@@ -101,9 +101,7 @@ class Server:
 
     def _listmethods(self, cl=None):
         if not cl: cl = self.__class__
-        names = list(cl.__dict__.keys())
-        names = [x for x in names if x[0] != '_']
-        names.sort()
+        names = sorted([x for x in cl.__dict__.keys() if x[0] != '_'])
         for base in cl.__bases__:
             basenames = self._listmethods(base)
             basenames = list(filter(lambda x, names=names: x not in names, basenames))
index d0a08249dcd50aa420726e3d357fcde36d1f7057..19ad3928ccddaf9bf8be88cfeafa5ee1a91f9af5 100755 (executable)
@@ -106,9 +106,7 @@ def showbar(dict, title):
     n = len(title)
     print('='*((70-n)/2), title, '='*((71-n)/2))
     list = []
-    keys = list(dict.keys())
-    keys.sort()
-    for key in keys:
+    for key in sorted(dict.keys()):
         n = len(str(key))
         list.append((len(dict[key]), key))
     maxkeylength = 0
@@ -128,8 +126,7 @@ def show(dict, title, maxitems):
     n = len(title)
     print('='*((70-n)/2), title, '='*((71-n)/2))
     list = []
-    keys = list(dict.keys())
-    for key in keys:
+    for key in dict.keys():
         list.append((-len(dict[key]), key))
     list.sort()
     for count, key in list[:maxitems]:
index a91e3594ef037bc678257cfcd3c322a0c4f2106d..3dc57ce63af67e5bef48a3181630e1c6a363c311 100755 (executable)
@@ -87,7 +87,7 @@ def test():
         return
     if debug: print('done.')
     if debug > 1:
-        for key in list(m.trans.keys()):
+        for key in m.trans.keys():
             if key is None or len(key) < histsize:
                 print(repr(key), m.trans[key])
         if histsize == 0: print(repr(''), m.trans[''])
index 4635f1d8d6e3c200ba9f2ca48471215a4be31b65..7038380f5c02553b46551c149e1133eeab3c4a85 100755 (executable)
@@ -172,10 +172,9 @@ def printtree(f, tree, indent, p):
         createpage(p[1:], tree, p)
         return
 
-    kl = list(tree.keys())
+    kl = sorted(tree.keys())
 
     if l > 1:
-        kl.sort()
         if indent > 0:
             # Create a sub-list
             f.write('<LI>'+p[1:]+'\n<UL>')
index 6bf788b0c82694a47d7a2134fe8ef6fb30cfe97d..4a155f89ae9ac96c399fd268454f5067aee095ac 100644 (file)
@@ -127,7 +127,7 @@ class Coroutine:
         if self.killed:
             raise TypeError('kill() called on dead coroutines')
         self.killed = 1
-        for coroutine in list(self.invokedby.keys()):
+        for coroutine in self.invokedby.keys():
             coroutine.resume()
 
     def back(self, data=None):
index 2cf625afcbfdc5f191a328a1f19266a248e797ab..1d39420cd9122b6d929ea82b360509fcd4f1ec18 100755 (executable)
@@ -40,7 +40,7 @@ def RunSample(w):
     #        global variables "demo_opt_from" and "demo_opt_to". Otherwise
     #        the OptionMenu widget will complain about "unknown options"!
     #
-    for opt in list(options.keys()):
+    for opt in options.keys():
         from_file.add_command(opt, label=options[opt])
         to_file.add_command(opt, label=options[opt])
 
index 98bdd145cbc26fa9d2d2f28a88b9d8964ac310dd..921c105ae26c4a1852f6bcb0d51f98feccd750bb 100755 (executable)
@@ -112,7 +112,7 @@ class Dialog:
     def addchoices(self):
         self.choices = {}
         list = []
-        for k, dc in list(self.options.items()):
+        for k, dc in self.options.items():
             list.append((k, dc))
         list.sort()
         for k, (d, c) in list:
@@ -260,7 +260,7 @@ class WidgetDialog(Dialog):
             classes = {}
             for c in (self.classes,
                       self.addclasses[self.klass]):
-                for k in list(c.keys()):
+                for k in c.keys():
                     classes[k] = c[k]
             self.classes = classes
 
@@ -273,7 +273,7 @@ class WidgetDialog(Dialog):
     def update(self):
         self.current = {}
         self.options = {}
-        for k, v in list(self.configuration.items()):
+        for k, v in self.configuration.items():
             if len(v) > 4:
                 self.current[k] = v[4]
                 self.options[k] = v[3], v[2] # default, klass
index a2b4647548d74501bfc22dfeb3852bcd24b2fd03..e083e641d44ff6b026c61c769d0a71cffde7239e 100644 (file)
@@ -15,7 +15,7 @@ class FancyCounter(handler.ContentHandler):
         self._attrs = self._attrs + len(attrs)
         self._elem_types[name] = self._elem_types.get(name, 0) + 1
 
-        for name in list(attrs.keys()):
+        for name in attrs.keys():
             self._attr_types[name] = self._attr_types.get(name, 0) + 1
 
     def endDocument(self):
@@ -23,11 +23,11 @@ class FancyCounter(handler.ContentHandler):
         print("There were", self._attrs, "attributes.")
 
         print("---ELEMENT TYPES")
-        for pair in  list(self._elem_types.items()):
+        for pair in  self._elem_types.items():
             print("%20s %d" % pair)
 
         print("---ATTRIBUTE TYPES")
-        for pair in  list(self._attr_types.items()):
+        for pair in  self._attr_types.items():
             print("%20s %d" % pair)
 
 
index b0e131e0995e93bb1a25411151126177d3e740fe..8d7d4374c1bcf184acbd2fac8795903b19567ac8 100644 (file)
@@ -22,7 +22,7 @@ class ContentGenerator(handler.ContentHandler):
 
     def startElement(self, name, attrs):
         self._out.write('<' + name)
-        for (name, value) in list(attrs.items()):
+        for (name, value) in attrs.items():
             self._out.write(' %s="%s"' % (name, saxutils.escape(value)))
         self._out.write('>')