diff options
| author | Chris McDonough <chrism@plope.com> | 2011-12-01 13:30:44 -0500 |
|---|---|---|
| committer | Chris McDonough <chrism@plope.com> | 2011-12-01 13:30:44 -0500 |
| commit | ae0ff29c220bf7f4cd253ae94855c3a33bf2a497 (patch) | |
| tree | 8f51c4b1b2b6871edb7d02e266b1b7713bee0d2f | |
| parent | 2cb381ceadc26ebc5cf51c28ecba572a05300d8a (diff) | |
| download | pyramid-ae0ff29c220bf7f4cd253ae94855c3a33bf2a497.tar.gz pyramid-ae0ff29c220bf7f4cd253ae94855c3a33bf2a497.tar.bz2 pyramid-ae0ff29c220bf7f4cd253ae94855c3a33bf2a497.zip | |
minor cleanups
| -rw-r--r-- | pyramid/config/__init__.py | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/pyramid/config/__init__.py b/pyramid/config/__init__.py index 31f35e5ff..ff0e3581d 100644 --- a/pyramid/config/__init__.py +++ b/pyramid/config/__init__.py @@ -993,6 +993,7 @@ class ActionState(object): kw = action['kw'] info = action['info'] introspectables = action['introspectables'] + try: if callable is not None: callable(*args, **kw) @@ -1006,6 +1007,7 @@ class ActionState(object): tb) finally: del t, v, tb + if introspector is not None: for introspectable in introspectables: introspectable.register(introspector, info) @@ -1054,17 +1056,18 @@ def resolveConflicts(actions): def bypath(action): return (action['includepath'], action['order']) dups.sort(key=bypath) - output.append(dups[0]) - basepath = dups[0]['includepath'] - baseinfo = dups[0]['info'] - discriminator = dups[0]['discriminator'] + first = dups[0] + output.append(first) + basepath = first['includepath'] + baseinfo = first['info'] + discriminator = first['discriminator'] for dup in dups[1:]: includepath = dup['includepath'] # Test whether path is a prefix of opath if (includepath[:len(basepath)] != basepath # not a prefix or includepath == basepath): - L = conflicts.setdefault(discriminator, [baseinfo]) - L.append(dup['info']) + infos = conflicts.setdefault(discriminator, [baseinfo]) + infos.append(dup['info']) if conflicts: raise ConfigurationConflictError(conflicts) |
