From 230feba5fd4ca3db18775d3cfa89e42284833aac Mon Sep 17 00:00:00 2001 From: Jeremy Hylton Date: Thu, 26 Mar 2009 16:56:59 +0000 Subject: urlencode: Remove dead code branch created by automated conversion from 2.x. Clean up a few comments. --- Lib/urllib/parse.py | 18 +++++------------- 1 file changed, 5 insertions(+), 13 deletions(-) diff --git a/Lib/urllib/parse.py b/Lib/urllib/parse.py index 044c810..5fd038e 100644 --- a/Lib/urllib/parse.py +++ b/Lib/urllib/parse.py @@ -521,18 +521,17 @@ def urlencode(query, doseq=0): """ if hasattr(query, "items"): - # mapping objects query = query.items() else: - # it's a bother at times that strings and string-like objects are - # sequences... + # It's a bother at times that strings and string-like objects are + # sequences. try: # non-sequence items should not work with len() # non-empty strings will fail this if len(query) and not isinstance(query[0], tuple): raise TypeError - # zero-length sequences of all types will get here and succeed, - # but that's a minor nit - since the original implementation + # Zero-length sequences of all types will get here and succeed, + # but that's a minor nit. Since the original implementation # allowed empty dicts that type of behavior probably should be # preserved for consistency except TypeError: @@ -542,7 +541,6 @@ def urlencode(query, doseq=0): l = [] if not doseq: - # preserve old behavior for k, v in query: k = quote_plus(str(k)) v = quote_plus(str(v)) @@ -553,15 +551,9 @@ def urlencode(query, doseq=0): if isinstance(v, str): v = quote_plus(v) l.append(k + '=' + v) - elif isinstance(v, str): - # is there a reasonable way to convert to ASCII? - # encode generates a string, but "replace" or "ignore" - # lose information and "strict" can raise UnicodeError - v = quote_plus(v.encode("ASCII", "replace")) - l.append(k + '=' + v) else: try: - # is this a sufficient test for sequence-ness? + # Is this a sufficient test for sequence-ness? x = len(v) except TypeError: # not a sequence -- cgit v0.12