From 06b49569b4e3d46a3517cb4241c09cb0998a8c5f 2010-09-27 08:03:39 From: Fernando Perez Date: 2010-09-27 08:03:39 Subject: [PATCH] Fix bug for x.*a*? syntax, added tests for it. --- diff --git a/IPython/core/inputsplitter.py b/IPython/core/inputsplitter.py index bdf1a37..b53d6bc 100644 --- a/IPython/core/inputsplitter.py +++ b/IPython/core/inputsplitter.py @@ -626,7 +626,7 @@ class InputSplitter(object): line_split = re.compile(""" ^(\s*) # any leading space ([,;/%]|!!?|\?\??) # escape character or characters - \s*(%?[\w\.]*) # function/method, possibly with leading % + \s*(%?[\w\.\*]*) # function/method, possibly with leading % # to correctly treat things like '?%magic' (\s+.*$|$) # rest of line """, re.VERBOSE) @@ -674,6 +674,8 @@ def split_user_input(line): ('', '', 'f.g', '(x)') >>> split_user_input('?%hist') ('', '?', '%hist', '') + >>> split_user_input('?x*') + ('', '?', 'x*', '') """ match = line_split.match(line) if match: @@ -850,6 +852,7 @@ class EscapedTransformer(object): # There may be one or two '?' at the end, move them to the front so that # the rest of the logic can assume escapes are at the start + l_ori = line_info line = line_info.line if line.endswith('?'): line = line[-1] + line[:-1] @@ -857,8 +860,11 @@ class EscapedTransformer(object): line = line[-1] + line[:-1] line_info = LineInfo(line) - # From here on, simply choose which level of detail to get. - if line_info.esc == '?': + # From here on, simply choose which level of detail to get, and + # special-case the psearch syntax + if '*' in line_info.line: + pinfo = 'psearch' + elif line_info.esc == '?': pinfo = 'pinfo' elif line_info.esc == '??': pinfo = 'pinfo2' diff --git a/IPython/core/tests/test_inputsplitter.py b/IPython/core/tests/test_inputsplitter.py index 34f66b4..fd2dbfc 100644 --- a/IPython/core/tests/test_inputsplitter.py +++ b/IPython/core/tests/test_inputsplitter.py @@ -451,6 +451,8 @@ syntax = \ ('x3?', 'get_ipython().magic("pinfo x3")'), ('x4??', 'get_ipython().magic("pinfo2 x4")'), ('%hist?', 'get_ipython().magic("pinfo %hist")'), + ('f*?', 'get_ipython().magic("psearch f*")'), + ('ax.*aspe*?', 'get_ipython().magic("psearch ax.*aspe*")'), ], # Explicit magic calls