Merge lp:~jorgen-stenarson/ipython/ipython-bug-315392 into lp:ipython/0.11

Proposed by Jörgen Stenarson
Status: Merged
Merge reported by: Brian Granger
Merged at revision: not available
Proposed branch: lp:~jorgen-stenarson/ipython/ipython-bug-315392
Merge into: lp:ipython/0.11
Diff against target: None lines
To merge this branch: bzr merge lp:~jorgen-stenarson/ipython/ipython-bug-315392
Reviewer Review Type Date Requested Status
IPython Developers Pending
Review via email: mp+5624@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Jörgen Stenarson (jorgen-stenarson) wrote :

This is a quickfix for a unicode problem in the readline completer delims initialization.

The idea is to just zap any non-ascii characters from the completer_delim string. This should work fine because current behaviour is to just allow strings anyway.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'IPython/iplib.py'
--- IPython/iplib.py 2009-04-07 07:59:15 +0000
+++ IPython/iplib.py 2009-04-16 17:37:40 +0000
@@ -1475,7 +1475,7 @@
1475 readline.parse_and_bind(rlcommand)1475 readline.parse_and_bind(rlcommand)
14761476
1477 # remove some chars from the delimiters list1477 # remove some chars from the delimiters list
1478 delims = readline.get_completer_delims()1478 delims = readline.get_completer_delims().encode("ascii", "ignore")
1479 delims = delims.translate(string._idmap,1479 delims = delims.translate(string._idmap,
1480 self.rc.readline_remove_delims)1480 self.rc.readline_remove_delims)
1481 readline.set_completer_delims(delims)1481 readline.set_completer_delims(delims)

Subscribers

People subscribed via source and target branches