------------------------------------------------------------
revno: 1537
committer: Mark Sapiro <m...@msapiro.net>
branch nick: 2.1
timestamp: Sat 2015-02-07 23:54:26 -0800
message:
Cleaned the code a bit. No change in results.
modified:
Mailman/versions.py
--
lp:mailman/2.1
https://code.launchpad.net/~mailman-coders/mailman/2.1
Your team Mailman Checkins is subscribed to branch lp:mailman/2.1.
To unsubscribe from this branch go to
https://code.launchpad.net/~mailman-coders/mailman/2.1/+edit-subscription
=== modified file 'Mailman/versions.py'
--- Mailman/versions.py 2015-02-07 20:44:08 +0000
+++ Mailman/versions.py 2015-02-08 07:54:26 +0000
@@ -110,28 +110,20 @@
"""Recursively process lists, tuples and dictionary values and
convert strings as needed. Return either the updated item or None
if no change."""
+ changed = False
if isinstance(v, str):
- nv = convert(v, f, t)
- if nv == v:
- return None
- else:
- return nv
+ return convert(v, f, t)
elif isinstance(v, list):
- changed = False
- nl = []
for i in range(len(v)):
nv = doitem(v[i], f, t)
if nv:
changed = True
- nl += [nv]
- else:
- nl += v[i]
+ v[i] = nv
if changed:
- return nl
+ return v
else:
return None
elif isinstance(v, tuple):
- changed = False
nt = ()
for i in range(len(v)):
nv = doitem(v[i], f, t)
@@ -145,17 +137,13 @@
else:
return None
elif isinstance(v, dict):
- changed = False
- nd = {}
for k, ov in v.items():
nv = doitem(ov, f, t)
if nv:
changed = True
- nd[k] = nv
- else:
- nd[k] = ov
+ v[k] = nv
if changed:
- return nd
+ return v
else:
return None
else:
@@ -178,7 +166,7 @@
if is_f and not is_t:
return u.encode(t, 'replace')
else:
- return s
+ return None
# Migrate to 2.1b3, baw 17-Aug-2001
if hasattr(l, 'dont_respond_to_post_requests'):
_______________________________________________
Mailman-checkins mailing list
Mailman-checkins@python.org
Unsubscribe:
https://mail.python.org/mailman/options/mailman-checkins/archive%40jab.org
