From: Jason Kirtland <jek@discorporate.us>
Date: Tue, 15 Jul 2008 19:29:41 +0000 (+0000)
Subject: - Dropped `reversed` emulation
X-Git-Tag: rel_0_5beta3~50
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=1d37472fdd1e4091fe2d158a21b27043fc8626f9;p=thirdparty%2Fsqlalchemy%2Fsqlalchemy.git

- Dropped `reversed` emulation
---

diff --git a/lib/sqlalchemy/orm/mapper.py b/lib/sqlalchemy/orm/mapper.py
index aec4e1dff5..c0fd1c550f 100644
--- a/lib/sqlalchemy/orm/mapper.py
+++ b/lib/sqlalchemy/orm/mapper.py
@@ -1175,7 +1175,7 @@ class Mapper(object):
                     # synchronize newly inserted ids from one table to the next
                     # TODO: this fires off more than needed, try to organize syncrules
                     # per table
-                    for m in util.reversed(list(mapper.iterate_to_root())):
+                    for m in reversed(list(mapper.iterate_to_root())):
                         if m.__inherits_equated_pairs:
                             m.__synchronize_inherited(state)
 
@@ -1562,7 +1562,7 @@ class Mapper(object):
         allconds = []
 
         start = False
-        for mapper in util.reversed(list(self.iterate_to_root())):
+        for mapper in reversed(list(self.iterate_to_root())):
             if mapper.local_table in tables:
                 start = True
             if start and not mapper.single:
diff --git a/lib/sqlalchemy/orm/unitofwork.py b/lib/sqlalchemy/orm/unitofwork.py
index 9a008e5cb1..b4e649262a 100644
--- a/lib/sqlalchemy/orm/unitofwork.py
+++ b/lib/sqlalchemy/orm/unitofwork.py
@@ -755,7 +755,7 @@ class UOWExecutor(object):
             for task in tasks:
                 self.execute_save_steps(trans, task)
         if isdelete is not False:
-            for task in util.reversed(tasks):
+            for task in reversed(tasks):
                 self.execute_delete_steps(trans, task)
 
     def save_objects(self, trans, task):
@@ -782,7 +782,7 @@ class UOWExecutor(object):
             for dep in task.polymorphic_dependencies:
                 self.execute_dependency(trans, dep, False)
         if isdelete is not False:
-            for dep in util.reversed(list(task.polymorphic_dependencies)):
+            for dep in reversed(list(task.polymorphic_dependencies)):
                 self.execute_dependency(trans, dep, True)
 
     def execute_cyclical_dependencies(self, trans, task, isdelete):
diff --git a/lib/sqlalchemy/orm/uowdumper.py b/lib/sqlalchemy/orm/uowdumper.py
index 09b82167da..43461885d4 100644
--- a/lib/sqlalchemy/orm/uowdumper.py
+++ b/lib/sqlalchemy/orm/uowdumper.py
@@ -6,7 +6,6 @@
 
 """Dumps out a string representation of a UOWTask structure"""
 
-from sqlalchemy import util
 from sqlalchemy.orm import unitofwork
 from sqlalchemy.orm import util as mapperutil
 
@@ -22,7 +21,7 @@ class UOWDumper(unitofwork.UOWExecutor):
             for task in tasks:
                 self._execute(trans, task, False)
         if isdelete is not False:
-            for task in util.reversed(tasks):
+            for task in reversed(tasks):
                 self._execute(trans, task, True)
 
     def _execute(self, trans, task, isdelete):
diff --git a/lib/sqlalchemy/sql/util.py b/lib/sqlalchemy/sql/util.py
index c097443937..66ad3d32d7 100644
--- a/lib/sqlalchemy/sql/util.py
+++ b/lib/sqlalchemy/sql/util.py
@@ -20,7 +20,7 @@ def sort_tables(tables, reverse=False):
         visitors.traverse(table, {'schema_visitor':True}, {'foreign_key':visit_foreign_key})    
     sequence = topological.sort(tuples, tables)
     if reverse:
-        return util.reversed(sequence)
+        return reversed(sequence)
     else:
         return sequence
 
diff --git a/lib/sqlalchemy/topological.py b/lib/sqlalchemy/topological.py
index bfcfc9c0ad..e46b41a492 100644
--- a/lib/sqlalchemy/topological.py
+++ b/lib/sqlalchemy/topological.py
@@ -18,7 +18,6 @@ conditions.
 
 """
 
-from sqlalchemy import util
 from sqlalchemy.exc import CircularDependencyError
 
 __all__ = ['sort', 'sort_with_cycles', 'sort_as_tree']
@@ -237,7 +236,7 @@ def _organize_as_tree(nodes):
     # order of the list has no semantics for the algorithmic
     independents = []
     # in reverse topological order
-    for node in util.reversed(nodes):
+    for node in reversed(nodes):
         # nodes subtree and cycles contain the node itself
         subtree = set([node])
         if node.cycles is not None:
diff --git a/lib/sqlalchemy/util.py b/lib/sqlalchemy/util.py
index a100d931d3..ee2840cf50 100644
--- a/lib/sqlalchemy/util.py
+++ b/lib/sqlalchemy/util.py
@@ -27,16 +27,6 @@ try:
 except ImportError:
     import pickle
 
-try:
-    reversed = __builtin__.reversed
-except AttributeError:
-    def reversed(seq):
-        i = len(seq) -1
-        while  i >= 0:
-            yield seq[i]
-            i -= 1
-        raise StopIteration()
-
 try:
     # Try the standard decimal for > 2.3 or the compatibility module
     # for 2.3, if installed.