Re: segfault with nfa

2014-08-27 Fir de Conversatie mattn
I reproduced it. -- -- You received this message from the vim_dev maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups vim_dev

Re: segfault with nfa

2014-08-27 Fir de Conversatie mattn
Seems need to avoid following by zero-width. diff -r aeafe78fc869 src/regexp_nfa.c --- a/src/regexp_nfa.c Fri Aug 22 13:13:28 2014 +0200 +++ b/src/regexp_nfa.c Wed Aug 27 15:35:21 2014 +0900 @@ -4408,8 +4408,10 @@ case NFA_SPLIT: /* order matters here */ - subs =

Update doc/eval.txt about expand()

2014-08-27 Fir de Conversatie mattn
See: http://www.reddit.com/r/vim/comments/2el2zo/til_expandshellvar_might_spawn_a_shell/ IFAIK, expand('$FOO') doesn't spawn shell in latest version of vim. So I think it need to update doc. https://gist.github.com/mattn/18c27561455271d17158 Thanks. - Yasuhiro Matsumoto -- -- You received

Re: Update doc/eval.txt about expand()

2014-08-27 Fir de Conversatie LCD 47
On 27 August 2014, mattn mattn...@gmail.com wrote: See: http://www.reddit.com/r/vim/comments/2el2zo/til_expandshellvar_might_spawn_a_shell/ IFAIK, expand('$FOO') doesn't spawn shell in latest version of vim. [...] AFAICT it still does, please see os_unix.c lines 5755--6228. /lcd

Re: Update doc/eval.txt about expand()

2014-08-27 Fir de Conversatie mattn
Yes, I know. But in this case, afaik, expand('$FOO') shouldn't call mch_expand_wildcards(). -- -- You received this message from the vim_dev maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You

Re: Update doc/eval.txt about expand()

2014-08-27 Fir de Conversatie mattn
On Wednesday, August 27, 2014 9:58:15 PM UTC+9, mattn wrote: Yes, I know. But in this case, afaik, expand('$FOO') shouldn't call mch_expand_wildcards(). Ooops, sorry. I found it. https://code.google.com/p/vim/source/browse/src/misc1.c#10769 -- -- You received this message from the vim_dev

Re: [patch] Fix netrw#NetrwBrowseX with `kdeinit` running, but no `kfmclient`

2014-08-27 Fir de Conversatie Charles Campbell
Daniel Hahler wrote: When there is a `kdeinit4` process, but `kfmclient` is not installed `BrowseX` fails to open an URL (it falls through to the last else block). This happened on a non-KDE system, where `kdeinit4` was installed/started for an unknown reason, but konqueror/kfmclient is not

getting an up-to-date runtime

2014-08-27 Fir de Conversatie Charles E Campbell
Hello: I don't have mercurial on this machine -- we use cvs. Consequently, what I'd used in the past to update runtime was: rsync -avzcP --delete --exclude=dos --exclude=spell ftp.nluug.nl::Vim/runtime/ . but, having tried it this morning, I get: rsync: failed to connect to

Re: getting an up-to-date runtime

2014-08-27 Fir de Conversatie Bram Moolenaar
Charles Campbell wrote: I don't have mercurial on this machine -- we use cvs. Consequently, what I'd used in the past to update runtime was: rsync -avzcP --delete --exclude=dos --exclude=spell ftp.nluug.nl::Vim/runtime/ . but, having tried it this morning, I get: rsync:

Re: 7.4.356 - bad patch on ftp

2014-08-27 Fir de Conversatie Bram Moolenaar
Arkadiusz Miśkiewicz wrote: On Tuesday 22 of July 2014, Bram Moolenaar wrote: Arkadiusz Miśkiewicz wrote: Could someone fix 7.4.356 patch that sits on vim ftp? It tries to patch .hgignore file which doesn't exist in released tarballs. OK, let's do the same as for the other patch

Re: Update doc/eval.txt about expand()

2014-08-27 Fir de Conversatie Bram Moolenaar
Yasuhiro Matsumoto wrote: See: http://www.reddit.com/r/vim/comments/2el2zo/til_expandshellvar_might_spawn_a_shell/ IFAIK, expand('$FOO') doesn't spawn shell in latest version of vim. So I think it need to update doc. https://gist.github.com/mattn/18c27561455271d17158 Thanks! --

Re: segfault with nfa

2014-08-27 Fir de Conversatie Bram Moolenaar
Yasuhiro Matsumoto wrote: Seems need to avoid following by zero-width. diff -r aeafe78fc869 src/regexp_nfa.c --- a/src/regexp_nfa.cFri Aug 22 13:13:28 2014 +0200 +++ b/src/regexp_nfa.cWed Aug 27 15:35:21 2014 +0900 @@ -4408,8 +4408,10 @@ case NFA_SPLIT:

Re: [patch] Fix netrw#NetrwBrowseX with `kdeinit` running, but no `kfmclient`

2014-08-27 Fir de Conversatie Daniel Hahler
Am Mittwoch, 27. August 2014 16:40:41 UTC+2 schrieb Charles Campbell: Daniel Hahler wrote: When there is a `kdeinit4` process, but `kfmclient` is not installed `BrowseX` fails to open an URL (it falls through to the last else block). This happened on a non-KDE system, where

:set all doesn't reset indent amount for cindent.

2014-08-27 Fir de Conversatie Yukihiro Nakadaira
:set all doesn't reset indent amount for cindent. Steps to reproduce: $ vim -u NONE :set shiftwidth=2 :set all :set shiftwidth? shiftwidth=8 :set cindent i{ This line should be indented with 8 spaces (tab character). }Esc Result: { This line should be indented with 8 spaces