Fixed bug on __getitem__ (infinite loop because pop() use it already)

1 files changed, 3 insertions, 2 deletions

diff --git a/sqlparse/utils.py b/sqlparse/utils.py
index 016a292..cdf27b1 100644
--- a/sqlparse/utils.py
+++ b/sqlparse/utils.py
@@ -20,8 +20,9 @@ if OrderedDict:
             self._maxsize = maxsize
 
         def __getitem__(self, key, *args, **kwargs):
-            # Remove the (key, value) pair from the cache, or raise KeyError
-            value = self.pop(key)
+            # Get the key and remove it from the cache, or raise KeyError
+            value = OrderedDict.__getitem__(self, key)
+            del self[key]
 
             # Insert the (key, value) pair on the front of the cache
             OrderedDict.__setitem__(self, key, value)