[Python-checkins] cpython (3.2): Back out fix for issue #13886; it introduced a new bug in interactive readline

nadeem.vawda python-checkins at python.org
Sat Feb 2 20:05:20 CET 2013


http://hg.python.org/cpython/rev/e6952acd5a55
changeset:   81943:e6952acd5a55
branch:      3.2
parent:      81938:434b50c7bbed
user:        Nadeem Vawda <nadeem.vawda at gmail.com>
date:        Sat Feb 02 20:05:11 2013 +0100
summary:
  Back out fix for issue #13886; it introduced a new bug in interactive readline use.

files:
  Lib/test/test_builtin.py |  7 -------
  Misc/NEWS                |  4 ----
  Modules/readline.c       |  2 +-
  3 files changed, 1 insertions(+), 12 deletions(-)


diff --git a/Lib/test/test_builtin.py b/Lib/test/test_builtin.py
--- a/Lib/test/test_builtin.py
+++ b/Lib/test/test_builtin.py
@@ -18,13 +18,6 @@
     import pty, signal
 except ImportError:
     pty = signal = None
-# Importing this module has the side-effect of changing the behavior of input().
-# Ensure that we always use the readline version (if available), so we don't get
-# different results depending on what other tests have already imported.
-try:
-    import readline
-except ImportError:
-    pass
 
 
 class Squares:
diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -12,10 +12,6 @@
 
 - Issue #16979: Fix error handling bugs in the unicode-escape-decode decoder.
 
-- Issue #13886: Fix input() to not strip out input bytes that cannot be decoded
-  using the locale encoding. Also fix sporadic failures in test_builtin due to
-  dependence on whether the readline module has previously been imported.
-
 - Issue #10156: In the interpreter's initialization phase, unicode globals
   are now initialized dynamically as needed.
 
diff --git a/Modules/readline.c b/Modules/readline.c
--- a/Modules/readline.c
+++ b/Modules/readline.c
@@ -1068,7 +1068,7 @@
     char *saved_locale = strdup(setlocale(LC_CTYPE, NULL));
     if (!saved_locale)
         Py_FatalError("not enough memory to save locale");
-    setlocale(LC_CTYPE, "C");
+    setlocale(LC_CTYPE, "");
 #endif
 
     if (sys_stdin != rl_instream || sys_stdout != rl_outstream) {

-- 
Repository URL: http://hg.python.org/cpython


More information about the Python-checkins mailing list