[IPython-dev] Release plans...

Dan Milstein danmil@comcast....
Thu Apr 5 18:18:55 CDT 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