[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: address@hidden: Re: comint's directory tracking doesn't understand \
From: |
David Hansen |
Subject: |
Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)] |
Date: |
Sat, 10 Mar 2007 09:06:44 +0100 |
User-agent: |
Gnus/5.110006 (No Gnus v0.6) Emacs/22.0.95 (gnu/linux) |
On Fri, 09 Mar 2007 19:04:39 -0500 Chong Yidong wrote:
> I think the following patch fixes the same bug in a safer way.
>
> *** emacs/lisp/comint.el.~1.358.~ 2007-02-27 19:43:48.000000000 -0500
> --- emacs/lisp/comint.el 2007-03-09 18:44:15.000000000 -0500
> ***************
> *** 1384,1390 ****
> (let* ((first (if (if (fboundp 'w32-shell-dos-semantics)
> (w32-shell-dos-semantics))
> "[^ \n\t\"'`]+\\|"
> ! "[^ \n\t\"'`\\]+\\|\\\\[\"'`\\ \t]+\\|"))
> (argpart (concat first
> "\\(\"\\([^\"\\]\\|\\\\.\\)*\"\\|\
> '[^']*'\\|\
> --- 1384,1390 ----
> (let* ((first (if (if (fboundp 'w32-shell-dos-semantics)
> (w32-shell-dos-semantics))
> "[^ \n\t\"'`]+\\|"
> ! "\\([^ \n\t\"'`\\]\\|\\\\.\\)+\\|"))
> (argpart (concat first
> "\\(\"\\([^\"\\]\\|\\\\.\\)*\"\\|\
> '[^']*'\\|\
No it doesn't. (BTW that was my first suggestion). The string gets
still splitted at members of `comint-delimiter-argument-list' (note
that it's buffer local) no matter if they are escaped or not.
I have to admit that it probably can be fixed with far less than a
200 line patch but I'm not willing to touch these regexps and i
don't think it will result in anything "safer".
David
- Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)], (continued)
- Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)], David Hansen, 2007/03/05
- Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)], Richard Stallman, 2007/03/06
- Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)], Miles Bader, 2007/03/06
- Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)], David Hansen, 2007/03/07
- Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)], Richard Stallman, 2007/03/07
- Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)], Chong Yidong, 2007/03/09
- Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)], Glenn Morris, 2007/03/09
- Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)], David Hansen, 2007/03/09
- Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)], David Kastrup, 2007/03/09
- Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)], Chong Yidong, 2007/03/09
- Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)],
David Hansen <=
- Re: address@hidden: Re: comint's directory tracking doesn't understand \( or \)], Chong Yidong, 2007/03/10