[IPython-dev] Release plans...
Dan Milstein
danmil at comcast.net
Thu Apr 5 19:18:55 EDT 2007
>
> I just applied this to my local tree:
>
> Index: test_prefilter.py
> ===================================================================
> --- test_prefilter.py (revision 2218)
> +++ test_prefilter.py (working copy)
> @@ -145,7 +145,7 @@
> ( '?thing', handle_help, ),
> ( 'thing?', handle_help ), # '?' can trail...
> ( 'thing!', handle_normal), # but only '?' can trail
> - ( '!thing?', handle_help), # trailing '?' wins if more
> than one
> + ( '!thing?', handle_shell_escape), # trailing '?' wins if
> more than one
> ( ' ?thing', handle_help), # ignore leading whitespace
> ( '!ls', handle_shell_escape ),
> ( '%magic', handle_magic),
>
>
> I think that's the correct test in light of Stefan's patches and the
> behavior we want, no?
Yep, that's correct -- and I've actually added a whole set of further
tests in my private copy to ensure that it does the right thing for
all trailing qmark / leading esc char combos:
# Trailing qmark combos. Odd special cases abound
( '!thing?', handle_shell_escape), # trailing '?' loses to
shell esc
( '!thing ?', handle_shell_escape),
( '!!thing?', handle_shell_escape),
( '%cmd?', handle_help),
( '/cmd?', handle_help),
( ';cmd?', handle_help),
( ',cmd?', handle_help),
-D
More information about the IPython-dev
mailing list