Code modernization. Replace v=s[i]; del s[i] with single lookup v=s.pop(i)

diff --git a/Lib/Queue.py b/Lib/Queue.py
index de7be72d7a9d985db61f77cb4d6c91c5711fbdff..cd035da14cf419daa3376b1c03a0180ef8691b08 100644 (file)
--- a/Lib/Queue.py
+++ b/Lib/Queue.py
@@ -146,6 +146,4 @@ class Queue:
 
     # Get an item from the queue
     def _get(self):
-        item = self.queue[0]
-        del self.queue[0]
-        return item
+        return self.queue.pop(0)

diff --git a/Lib/asynchat.py b/Lib/asynchat.py
index e1c97949b00b0cbf7a897891c79ed32a7e121394..1ba6b15b911116c1bede769aa15baf3e4ce28b38 100644 (file)
--- a/Lib/asynchat.py
+++ b/Lib/asynchat.py
@@ -269,9 +269,7 @@ class fifo:
 
     def pop (self):
         if self.list:
-            result = self.list[0]
-            del self.list[0]
-            return (1, result)
+            return (1, self.list.pop(0))
         else:
             return (0, None)
 

diff --git a/Lib/cgi.py b/Lib/cgi.py
index 88d419e0ddf056c13713f7018887ec4c248a182c..5f4bad3c8b6e126162c50bd72fad58774825ecba 100755 (executable)
--- a/Lib/cgi.py
+++ b/Lib/cgi.py
@@ -323,8 +323,7 @@ def parse_header(line):
 
     """
     plist = map(lambda x: x.strip(), line.split(';'))
-    key = plist[0].lower()
-    del plist[0]
+    key = plist.pop(0).lower()
     pdict = {}
     for p in plist:
         i = p.find('=')

diff --git a/Lib/cmd.py b/Lib/cmd.py
index f20c1bb7aa7296ec0610bccdcebd14947b5d5fe9..6d9efff080069dc080cf06d6b0cdc4e9ba6b3845 100644 (file)
--- a/Lib/cmd.py
+++ b/Lib/cmd.py
@@ -109,8 +109,7 @@ class Cmd:
         stop = None
         while not stop:
             if self.cmdqueue:
-                line = self.cmdqueue[0]
-                del self.cmdqueue[0]
+                line = self.cmdqueue.pop(0)
             else:
                 if self.use_rawinput:
                     try:
@@ -261,11 +260,10 @@ class Cmd:
         names = []
         classes = [self.__class__]
         while classes:
-            aclass = classes[0]
+            aclass = classes.pop(0)
             if aclass.__bases__:
                 classes = classes + list(aclass.__bases__)
             names = names + dir(aclass)
-            del classes[0]
         return names
 
     def complete_help(self, *args):

diff --git a/Lib/imaplib.py b/Lib/imaplib.py
index 0817da9d23720a5b5a1cf3655ee10b00d9d19cbc..ed03c56e41b3be396b29ee79614531ded06f2994 100644 (file)
--- a/Lib/imaplib.py
+++ b/Lib/imaplib.py
@@ -973,11 +973,10 @@ class IMAP4:
             return typ, dat
         if not name in self.untagged_responses:
             return typ, [None]
-        data = self.untagged_responses[name]
+        data = self.untagged_responses.pop(name)
         if __debug__:
             if self.debug >= 5:
                 self._mesg('untagged_responses[%s] => %s' % (name, data))
-        del self.untagged_responses[name]
         return typ, data
 
 

diff --git a/Lib/mailbox.py b/Lib/mailbox.py
index 7afff50012b857095f7b90b02f54fe99ab9b6c8e..1834177f701e8e8cac11021aa3c754ea9cbcaee2 100755 (executable)
--- a/Lib/mailbox.py
+++ b/Lib/mailbox.py
@@ -202,8 +202,7 @@ class MHMailbox:
     def next(self):
         if not self.boxes:
             return None
-        fn = self.boxes[0]
-        del self.boxes[0]
+        fn = self.boxes.pop(0)
         fp = open(os.path.join(self.dirname, fn))
         return self.factory(fp)
 
@@ -233,8 +232,7 @@ class Maildir:
     def next(self):
         if not self.boxes:
             return None
-        fn = self.boxes[0]
-        del self.boxes[0]
+        fn = self.boxes.pop(0)
         fp = open(fn)
         return self.factory(fp)
 

diff --git a/Lib/multifile.py b/Lib/multifile.py
index ff7dbf6b14b39dd262d537f059301cf965ef9061..4ef364451dbd1e7ce60086c2e4b89cf8c0a561f2 100644 (file)
--- a/Lib/multifile.py
+++ b/Lib/multifile.py
@@ -160,8 +160,7 @@ class MultiFile:
         self.level = max(0, self.level - 1)
         del self.stack[0]
         if self.seekable:
-            self.start = self.posstack[0]
-            del self.posstack[0]
+            self.start = self.posstack.pop(0)
             if self.level > 0:
                 self.lastpos = abslastpos - self.start
 

diff --git a/Lib/mutex.py b/Lib/mutex.py
index 47d2ca23713caf5cdd4841b0a306fd139a2f3c85..e15710a908acf347c0e5307a3fc21139ac65ae08 100644 (file)
--- a/Lib/mutex.py
+++ b/Lib/mutex.py
@@ -44,8 +44,7 @@ class mutex:
         """Unlock a mutex.  If the queue is not empty, call the next
         function with its argument."""
         if self.queue:
-            function, argument = self.queue[0]
-            del self.queue[0]
+            function, argument = self.queue.pop(0)
             function(argument)
         else:
             self.locked = 0

diff --git a/Lib/pickle.py b/Lib/pickle.py
index f29df510a27d1defd3099fbb21d9dbd43caed5a9..e553920b12cd8044ad6e5b59390e896c75be1368 100644 (file)
--- a/Lib/pickle.py
+++ b/Lib/pickle.py
@@ -692,11 +692,7 @@ class Unpickler:
     dispatch[PERSID] = load_persid
 
     def load_binpersid(self):
-        stack = self.stack
-
-        pid = stack[-1]
-        del stack[-1]
-
+        pid = self.stack.pop()
         self.append(self.persistent_load(pid))
     dispatch[BINPERSID] = load_binpersid
 
@@ -977,8 +973,7 @@ class Unpickler:
 
     def load_append(self):
         stack = self.stack
-        value = stack[-1]
-        del stack[-1]
+        value = stack.pop()
         list = stack[-1]
         list.append(value)
     dispatch[APPEND] = load_append
@@ -995,9 +990,8 @@ class Unpickler:
 
     def load_setitem(self):
         stack = self.stack
-        value = stack[-1]
-        key = stack[-2]
-        del stack[-2:]
+        value = stack.pop()
+        key = stack.pop()
         dict = stack[-1]
         dict[key] = value
     dispatch[SETITEM] = load_setitem
@@ -1014,8 +1008,7 @@ class Unpickler:
 
     def load_build(self):
         stack = self.stack
-        value = stack[-1]
-        del stack[-1]
+        value = stack.pop()
         inst = stack[-1]
         try:
             setstate = inst.__setstate__
@@ -1038,8 +1031,7 @@ class Unpickler:
     dispatch[MARK] = load_mark
 
     def load_stop(self):
-        value = self.stack[-1]
-        del self.stack[-1]
+        value = self.stack.pop()
         raise _Stop(value)
     dispatch[STOP] = load_stop
 

diff --git a/Lib/threading.py b/Lib/threading.py
index 491a7c0b090edbb8a1e935c5db0c9a33a433ab80..0af193b8fa677af118dfc6d531d78f465f8219e8 100644 (file)
--- a/Lib/threading.py
+++ b/Lib/threading.py
@@ -633,8 +633,7 @@ def _test():
             while not self.queue:
                 self._note("get(): queue empty")
                 self.rc.wait()
-            item = self.queue[0]
-            del self.queue[0]
+            item = self.queue.pop(0)
             self._note("get(): got %s, %d left", item, len(self.queue))
             self.wc.notify()
             self.mon.release()

diff --git a/Lib/xmlrpclib.py b/Lib/xmlrpclib.py
index 0eb9f3f07396cfb75ab62f50188e61820159c28b..4b6fc43cff4c5cf1a6a81b461b20864c699e1700 100644 (file)
--- a/Lib/xmlrpclib.py
+++ b/Lib/xmlrpclib.py
@@ -779,16 +779,14 @@ class Unmarshaller:
     dispatch["name"] = end_string # struct keys are always strings
 
     def end_array(self, data):
-        mark = self._marks[-1]
-        del self._marks[-1]
+        mark = self._marks.pop()
         # map arrays to Python lists
         self._stack[mark:] = [self._stack[mark:]]
         self._value = 0
     dispatch["array"] = end_array
 
     def end_struct(self, data):
-        mark = self._marks[-1]
-        del self._marks[-1]
+        mark = self._marks.pop()
         # map structs to Python dictionaries
         dict = {}
         items = self._stack[mark:]