No need to handle want_string, I think.
https://gist.github.com/mattn/7192570202d96e5f8539
- mattn
--
--
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
---
Below is a patch to fix tv_equal.
https://gist.github.com/mattn/fb486829628b39e371a7
- mattn
--
--
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 r
Unfortunately, I got the same experience that Manuel Ortega went through.
Yesterday, I ran "make test" several times after pulling the patch into my
repo, and didn't see any failure with it.
But today, I got it (FWIW, after pulling other new patches into the repo
and building vim with them).
The
Thanks. I prefer to make difference for default and 1 clearly.
diff --git a/src/testdir/test_sort.vim b/src/testdir/test_sort.vim
index 819514a..25a6cea 100644
--- a/src/testdir/test_sort.vim
+++ b/src/testdir/test_sort.vim
@@ -38,8 +38,9 @@ endfunc
func Test_sort_default()
" docs say omitte
On Mon, Feb 29, 2016 at 09:38:42PM +0100, Bram Moolenaar wrote:
>
> Danek Duvall wrote:
>
> > Problems on Solaris after this patch, too. At least in the compilation
> > mode I've been using, isinf is not found by autoconf (though isnan is). So
> > we come into this section of macros.h with HAV
Patch 7.4.1467
Problem:Can't build without the float feature.
Solution: Add #ifdefs. (Nick Owens, closes #667)
Files: src/eval.c, src/json.c
*** ../vim-7.4.1466/src/eval.c 2016-02-29 22:05:16.841676901 +0100
--- src/eval.c 2016-02-29 23:10:08.812543404 +0100
***
*** 8415
Patch 7.4.1466
Problem:Coverity reports dead code.
Solution: Remove the two lines.
Files: src/channel.c
*** ../vim-7.4.1465/src/channel.c 2016-02-29 22:55:51.693608630 +0100
--- src/channel.c 2016-02-29 23:02:26.917430198 +0100
***
*** 1602,1609
Patch 7.4.1465
Problem:Coverity reported possible use of NULL pointer when using buffer
output with JSON mode.
Solution: Make it actually possible to use JSON mode with a buffer.
Re-encode the JSON to append it to the buffer.
Files: src/channel.c, src/testdir/tes
Patch 7.4.1464
Problem:When the argument of sort() is zero or empty it fails.
Solution: Make zero work as documented. (suggested by Yasuhiro Matsumoto)
Files: src/eval.c, src/testdir/test_sort.vim
*** ../vim-7.4.1463/src/eval.c 2016-02-29 21:05:43.539766722 +0100
--- src/eval.c
John Marriott wrote;
> On 28-Feb-2016 7:27 AM, Bram Moolenaar wrote:
> > Patch 7.4.1440 (after 7.4.1437)
> > Problem:Can't build on Windows.
> > Solution: Change #ifdefs. Only define isnan when used.
> > Files: src/macros.h, src/eval.c, src/json.c
> >
> >
> >
> I get this build failur
> help sort() says:
>
> > When {func} is omitted, is empty or zero, then sort() uses the
> > string representation of each item to sort on. Numbers sort
> > after Strings, |Lists| after Numbers. For sorting text in the
> > current buffer use |:sort|.
>
> But not work with zero.
Thanks. But tha
Patch 7.4.1463
Problem:Configure doesn't find isinf() and isnan() on some systems.
Solution: Use a configure check that includes math.h.
Files: src/configure.in, src/auto/configure
*** ../vim-7.4.1462/src/configure.in2016-02-27 20:14:09.920546700 +0100
--- src/configure.in2016
Danek Duvall wrote:
> Problems on Solaris after this patch, too. At least in the compilation
> mode I've been using, isinf is not found by autoconf (though isnan is). So
> we come into this section of macros.h with HAVE_ISNAN defined, but not
> HAVE_ISINF. Since they're both supposed to be mac
Patch 7.4.1462
Problem:Two more rarily used functions with errors.
Solution: Add proper argument types. (Dominique Pelle)
Files: src/misc2.c, src/termlib.c
*** ../vim-7.4.1461/src/misc2.c 2016-02-23 14:52:31.885232171 +0100
--- src/misc2.c 2016-02-29 21:17:47.323872879 +0100
*
Dominique wrote:
> Ken Takata wrote:
>
> > Hi Bram,
> >
> > 2016/1/31 Sun 4:31:47 UTC+9 Bram Moolenaar wrote:
> >> Patch 7.4.1213
> >> Problem:Using old style function declarations.
> >> Solution: Change to new style function declarations. (script by Hirohito
> >> Higashi)
> >
Josh Reichardt wrote:
>
> With no plugins installed:
>
> With the cursor before the |--install| flag I get |shConditional|.
> With the cursor after the |--install| flag I get |shCaseEsac|.
>
>
shCaseEsac should only be in effect when one is using a case construct.
Your example doesn't have a case
Yasuhiro Matsumoto wrote:
> job_start quote with double-quote even though no needed.
>
> job_start("cmd", "/c", "dir *.c")
>
> This should be
>
> cmd /c "dir *.c"
>
> But
>
> "cmd" "/c" "dir *.c"
>
> All arguments are quoted.
Thanks. It's probably good enough for now, but plea
Patch 7.4.1461
Problem:When starting job on MS-Windows all parts of the command are put
in quotes.
Solution: Only use quotes when needed. (Yasuhiro Matsumoto)
Files: src/eval.c
*** ../vim-7.4.1460/src/eval.c 2016-02-28 19:28:55.073515550 +0100
--- src/eval.c 2016-02-29 2
Manuel Ortega wrote:
> > > I just build 7.4.1457 on OS X 10.11.3, and `make test` failed with this
> > > error:
> > >
> > > From test_channel.vim:
> > > Found errors in Test_open_delay():
> > > function Test_open_delay[3]..3_run_server line 46: 'Caught
> > exception:
> > > Vim(call):E631: ch_eval
Problems on Solaris after this patch, too. At least in the compilation
mode I've been using, isinf is not found by autoconf (though isnan is). So
we come into this section of macros.h with HAVE_ISNAN defined, but not
HAVE_ISINF. Since they're both supposed to be macros in math.h, changing
i
Christian Brabandt wrote:
>
> Well, what is the output of the synID command given above? It does not
> seem reproducible.
>
>
Version questions:
* what version of syntax/sh.vim are you using? The latest release: v145
* what version of vim are you using? You need 7.4.1141 or later.
Charles Cam
On 28-Feb-2016 7:27 AM, Bram Moolenaar wrote:
Patch 7.4.1440 (after 7.4.1437)
Problem:Can't build on Windows.
Solution: Change #ifdefs. Only define isnan when used.
Files: src/macros.h, src/eval.c, src/json.c
I get this build failure on HP-UX after this patch:
cc -c -I. -Iprot
For the record, a version of this was merged into neovim:
https://github.com/neovim/neovim/pull/4304
--
--
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
---
On Monday, February 29, 2016 at 5:46:25 PM UTC+9, mattn wrote:
> job_start quote with double-quote even though no needed.
>
> job_start("cmd", "/c", "dir *.c")
Sorry, I meant job_start(["cmd", "/c", "dir *.c"])
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! T
Ken Takata wrote:
> 2016/2/29 Mon 6:34:08 UTC+9 Bram Moolenaar wrote:
> > Patch 7.4.1459 (after 7.4.1457)
> > Problem:MS-Windows doesn't know socklen_t.
> > Solution: Use previous method for WIN32.
> > Files: src/channel.c
>
> I don't think we should use previous method for Win32.
> P
Patch 7.4.1460
Problem:Syntax error in rarily used code.
Solution: Fix the mch_rename() declaration. (Ken Takata)
Files: src/os_unix.c, src/proto/os_unix.pro
*** ../vim-7.4.1459/src/os_unix.c 2016-02-27 18:13:05.236593109 +0100
--- src/os_unix.c 2016-02-29 12:47:50.8778524
Ken Takata wrote:
> Hi Bram,
>
> 2016/1/31 Sun 4:31:47 UTC+9 Bram Moolenaar wrote:
> > Patch 7.4.1213
> > Problem:Using old style function declarations.
> > Solution: Change to new style function declarations. (script by Hirohito
> > Higashi)
> > Files: src/os_amiga.c, src
Hi,
2016/2/29 Mon 6:34:08 UTC+9 Bram Moolenaar wrote:
> Patch 7.4.1459 (after 7.4.1457)
> Problem:MS-Windows doesn't know socklen_t.
> Solution: Use previous method for WIN32.
> Files: src/channel.c
I don't think we should use previous method for Win32.
Please check attached patch.
Re
Ken Takata wrote:
> Hi Bram,
>
> 2016/1/31 Sun 4:31:47 UTC+9 Bram Moolenaar wrote:
>> Patch 7.4.1213
>> Problem:Using old style function declarations.
>> Solution: Change to new style function declarations. (script by Hirohito
>> Higashi)
>> Files: src/os_amiga.c, src/os_ma
Hi Bram,
2016/1/31 Sun 4:31:47 UTC+9 Bram Moolenaar wrote:
> Patch 7.4.1213
> Problem:Using old style function declarations.
> Solution: Change to new style function declarations. (script by Hirohito
> Higashi)
> Files: src/os_amiga.c, src/os_mac_conv.c, src/os_msdos.d, src/
Hirohito Higashi wrote:
> 2016-2-22(Mon) 7:03:23 UTC+9 Bram Moolenaar:
> > Patch 7.4.1384
> > Problem:It is not easy to use a set of plugins and their dependencies.
> > Solution: Add packages, ":loadopt", 'packpath'.
>
> I found typo in the commit message.
>
> s/:loadopt/:loadplugin/
Oh,
Hmm, on my 10.11.3 environments, I cannot reproduce this failure...
--
--
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 becaus
job_start quote with double-quote even though no needed.
job_start("cmd", "/c", "dir *.c")
This should be
cmd /c "dir *.c"
But
"cmd" "/c" "dir *.c"
All arguments are quoted.
diff --git a/src/eval.c b/src/eval.c
index a208b2a..ab4e8ff 100644
--- a/src/eval.c
+++ b/src/eval.c
@@ -
33 matches
Mail list logo