diff options
author | Mike Bayer <mike_mp@zzzcomputing.com> | 2013-06-08 19:07:52 -0400 |
---|---|---|
committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2013-06-08 19:07:52 -0400 |
commit | a144f55e651427f73bfb62af09fec8041e548243 (patch) | |
tree | 3987e205099dc556f56a082d018376efc79a83cd | |
parent | 85368d25ed158c85bd19f4a63400884ab1cda26a (diff) | |
download | sqlalchemy-a144f55e651427f73bfb62af09fec8041e548243.tar.gz |
cleanup
-rw-r--r-- | lib/sqlalchemy/sql/util.py | 5 | ||||
-rw-r--r-- | lib/sqlalchemy/sql/visitors.py | 16 |
2 files changed, 6 insertions, 15 deletions
diff --git a/lib/sqlalchemy/sql/util.py b/lib/sqlalchemy/sql/util.py index bf3f3397e..c894357b5 100644 --- a/lib/sqlalchemy/sql/util.py +++ b/lib/sqlalchemy/sql/util.py @@ -812,11 +812,8 @@ class ClauseAdapter(visitors.ReplacingCloningVisitor): def __init__(self, selectable, equivalents=None, include=None, exclude=None, include_fn=None, exclude_fn=None, - adapt_on_names=False, - traverse_options=None): + adapt_on_names=False): self.__traverse_options__ = {'stop_on': [selectable]} - if traverse_options: - self.__traverse_options__.update(traverse_options) self.selectable = selectable if include: assert not include_fn diff --git a/lib/sqlalchemy/sql/visitors.py b/lib/sqlalchemy/sql/visitors.py index c5a45ffd4..24ada8d9f 100644 --- a/lib/sqlalchemy/sql/visitors.py +++ b/lib/sqlalchemy/sql/visitors.py @@ -256,12 +256,8 @@ def cloned_traverse(obj, opts, visitors): """clone the given expression structure, allowing modifications by visitors.""" - - if "cloned" in opts: - cloned = opts['cloned'] - else: - cloned = util.column_dict() - stop_on = util.column_set(opts.get('stop_on', [])) + cloned = {} + stop_on = set(opts.get('stop_on', [])) def clone(elem): if elem in stop_on: @@ -284,14 +280,12 @@ def replacement_traverse(obj, opts, replace): """clone the given expression structure, allowing element replacement by a given replacement function.""" - cloned = util.column_dict() - stop_on = util.column_set([id(x) for x in opts.get('stop_on', [])]) - unconditional = opts.get('unconditional', False) + cloned = {} + stop_on = set([id(x) for x in opts.get('stop_on', [])]) def clone(elem, **kw): if id(elem) in stop_on or \ - (not unconditional - and 'no_replacement_traverse' in elem._annotations): + 'no_replacement_traverse' in elem._annotations: return elem else: newelem = replace(elem) |