IPython

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

  1. ipython-bug-315392
  2. Merge into trunk
Proposed by Jörgen Stenarson
Status: Merged
Merge reported by: Brian Granger
Merged at revision: not available
Proposed branch: lp://qastaging/~jorgen-stenarson/ipython/ipython-bug-315392
Merge into: lp://qastaging/ipython/0.11
Diff against target: None lines
To merge this branch: bzr merge lp://qastaging/~jorgen-stenarson/ipython/ipython-bug-315392
Reviewer Review Type Date Requested Status
IPython Developers Pending
Review via email: mp+5624@code.qastaging.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
1=== modified file 'IPython/iplib.py'
2--- IPython/iplib.py 2009-04-07 07:59:15 +0000
3+++ IPython/iplib.py 2009-04-16 17:37:40 +0000
4@@ -1475,7 +1475,7 @@
5 readline.parse_and_bind(rlcommand)
6
7 # remove some chars from the delimiters list
8- delims = readline.get_completer_delims()
9+ delims = readline.get_completer_delims().encode("ascii", "ignore")
10 delims = delims.translate(string._idmap,
11 self.rc.readline_remove_delims)
12 readline.set_completer_delims(delims)

Subscribers

People subscribed via source and target branches