Re: [bug] Compiling vim.exe fails

2016-02-22 Fir de Conversatie Christian Brabandt
Am 2016-02-22 22:04, schrieb Bram Moolenaar: Christian Brabandt wrote: > > On Do, 18 Feb 2016, Bram Moolenaar wrote: > > > > > Christian Brabandt wrote: > > > > On Mi, 17 Feb 2016, Bram Moolenaar wrote: > > > > > > > > > Thanks. I think we should also build a console version. Since testing >

perleval bug on 64-bit big-endian architectures

2016-02-22 Fir de Conversatie Danek Duvall
I ran into a problem with a perl test on sparc. Looks like the code pulling keys out of a dict wasn't doing the right thing when it came to type declarations. Basically, on a 64-bit big-endian machine like SPARC, size_t will be 64 bits, and I32 will be 32 bits. If we pass the pointer to the 64-b

Re: job_start('echo hi') hangs on Linux, GTK2 GUI

2016-02-22 Fir de Conversatie tyru
2016/02/23 6:19 "Bram Moolenaar" : > > > Yasuhiro Matsumoto wrote: > > > Bram, it seems no one close the channel. > > > > diff --git a/src/channel.c b/src/channel.c > > index aa97088..0cf8718 100644 > > --- a/src/channel.c > > +++ b/src/channel.c > > @@ -1708,7 +1708,7 @@ channel_read(channel_T *ch

Re: Why "loadplugin" when there's already "loadplugins"?

2016-02-22 Fir de Conversatie Nikolay Aleksandrovich Pavlov
2016-02-22 22:17 GMT+03:00 Charles Campbell : > Bram Moolenaar wrote: >> Right, the name is not ideal, but I could not think of a better one. I >> would also use "bundle" instead of package or pack, but plugin >> managers already use it and there could be a conflict. I hope that the >> new mechanis

Patch 7.4.1397

2016-02-22 Fir de Conversatie Bram Moolenaar
Patch 7.4.1397 Problem:Sort test fails on MS-Windows. Solution: Correct the compare function. Files: src/testdir/test_sort.vim *** ../vim-7.4.1396/src/testdir/test_sort.vim 2016-02-22 22:51:27.521723233 +0100 --- src/testdir/test_sort.vim 2016-02-22 23:48:49.849510958 +0100 *

Patch 7.4.1396

2016-02-22 Fir de Conversatie Bram Moolenaar
Patch 7.4.1396 Problem:Compiler warnings for conversions. Solution: Add type cast. Files: src/ex_cmds2.c *** ../vim-7.4.1395/src/ex_cmds2.c 2016-02-22 20:18:57.109398389 +0100 --- src/ex_cmds2.c 2016-02-22 23:25:04.532479508 +0100 *** *** 3087,3094 if

Re: Why "loadplugin" when there's already "loadplugins"?

2016-02-22 Fir de Conversatie Justin M. Keyes
On Mon, Feb 22, 2016 at 2:17 PM, Charles Campbell wrote: > Bram Moolenaar wrote: >> Right, the name is not ideal, but I could not think of a better one. I >> would also use "bundle" instead of package or pack, but plugin >> managers already use it and there could be a conflict. I hope that the >>

Re: [PATCH] Compiler warning in ex_cmds2.c [after 7.4.1384]

2016-02-22 Fir de Conversatie Bram Moolenaar
Tux wrote: > Well, well. Sorry, I find it difficult to count beyond 4. :-) -- FATHER:You killed eight wedding guests in all! LAUNCELOT: Er, Well ... the thing is ... I thought your son was a lady. FATHER:I can understand that. "Monty Python and the Holy Grail" PYTHON

Patch 7.4.1395

2016-02-22 Fir de Conversatie Bram Moolenaar
Patch 7.4.1395 Problem:Using DETACH in quotes is not compatible with the Netbeans interface. (Xavier de Gaye) Solution: Remove the quotes, only use them for JSON and JS mode. Files: src/netbeans.c, src/channel.c *** ../vim-7.4.1394/src/netbeans.c 2016-02-20 18:18:51.7

Re: [Patch] Call sort() from within a comparison function

2016-02-22 Fir de Conversatie Bram Moolenaar
Jacob Niehus wrote: > Currently, the way sorting works in eval.c (global variables > representing the parameters of the current state) causes undefined > behavior in the event that a comparison function calls sort(). I moved > the parameters into a struct so it can be changed and reverted if > so

Patch 7.4.1394

2016-02-22 Fir de Conversatie Bram Moolenaar
Patch 7.4.1394 Problem:Can't sort inside a sort function. Solution: Use a struct to store the sort parameters. (Jacob Niehus) Files: src/eval.c, src/testdir/test_sort.vim *** ../vim-7.4.1393/src/eval.c 2016-02-22 21:48:26.813500402 +0100 --- src/eval.c 2016-02-22 22:48:37.471519676

Re: job_start('echo hi') hangs on Linux, GTK2 GUI

2016-02-22 Fir de Conversatie Bram Moolenaar
Yasuhiro Matsumoto wrote: > Bram, it seems no one close the channel. > > diff --git a/src/channel.c b/src/channel.c > index aa97088..0cf8718 100644 > --- a/src/channel.c > +++ b/src/channel.c > @@ -1708,7 +1708,7 @@ channel_read(channel_T *channel, int part, char *func) > > /* Reading a s

Patch 7.4.1393

2016-02-22 Fir de Conversatie Bram Moolenaar
Patch 7.4.1393 Problem:Starting a job hangs in the GUI. (Takuya Fujiwara) Solution: Don't check if ch_job is NULL when checking for an error. (Yasuhiro Matsumoto) Files: src/channel.c *** ../vim-7.4.1392/src/channel.c 2016-02-21 19:14:36.679958696 +0100 --- src/chann

Patch 7.4.1392

2016-02-22 Fir de Conversatie Bram Moolenaar
Patch 7.4.1392 Problem:Some tests fail for Win32 console version. Solution: Move the tests to SCRIPTS_MORE2. Pass VIMRUNTIME. (Christian Brabandt) Files: src/testdir/Make_all.mak *** ../vim-7.4.1391/src/testdir/Make_all.mak2016-02-02 23:22:38.101181704 +0100 --- src/

Re: [bug] Compiling vim.exe fails

2016-02-22 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > > > On Do, 18 Feb 2016, Bram Moolenaar wrote: > > > > > > > Christian Brabandt wrote: > > > > > On Mi, 17 Feb 2016, Bram Moolenaar wrote: > > > > > > > > > > > Thanks. I think we should also build a console version. Since > > > > > > testing > > > > > > appears to

Patch 7.4.1391

2016-02-22 Fir de Conversatie Bram Moolenaar
Patch 7.4.1391 Problem:Warning for uninitialzed variable. Solution: Set it to zero. (Christian Brabandt) Files: src/eval.c *** ../vim-7.4.1390/src/eval.c 2016-02-22 20:20:21.404530530 +0100 --- src/eval.c 2016-02-22 21:45:29.599359250 +0100 *** *** 9489,9495 s

Patch 7.4.1390

2016-02-22 Fir de Conversatie Bram Moolenaar
Patch 7.4.1390 Problem:When building with GTK and glib-compile-resources cannot be found building Vim fails. (Michael Gehring) Solution: Make GLIB_COMPILE_RESOURCES empty instead of leaving it at "no". (nuko8, closes #655) Files: src/configure.in, src/auto/config

Re: Solaris needs a tiny waittime, too

2016-02-22 Fir de Conversatie Danek Duvall
On Fri, Feb 19, 2016 at 02:56:35PM -0800, Danek Duvall wrote: > On Fri, Feb 19, 2016 at 11:21:46PM +0100, Bram Moolenaar wrote: > > > > > Danek Duvall wrote: > > > > > The channel tests, when run on a couple of the Solaris boxes I'm on, fail > > > fairly consistently on anywhere from two to six

Patch 7.4.1389

2016-02-22 Fir de Conversatie Bram Moolenaar
Patch 7.4.1389 Problem:Incomplete function declaration. Solution: Add "void". (Yasuhiro Matsumoto) Files: src/eval.c *** ../vim-7.4.1388/src/eval.c 2016-02-22 11:39:24.176069402 +0100 --- src/eval.c 2016-02-22 12:24:26.576161908 +0100 *** *** 15064,15070 * Called

Patch 7.4.1388

2016-02-22 Fir de Conversatie Bram Moolenaar
Patch 7.4.1388 Problem:Compiler warning. (Cesar Romani) Solution: Initialize variable. Files: src/ex_cmds2.c *** ../vim-7.4.1387/src/ex_cmds2.c 2016-02-21 23:01:47.441323390 +0100 --- src/ex_cmds2.c 2016-02-22 17:50:47.785044102 +0100 *** *** 3058,3071

Re: Warnings in ex_cmds2.c, getchar.c, pathdef.c, undo.c, os_win32.c

2016-02-22 Fir de Conversatie Bram Moolenaar
Dominique Pellé wrote: > "Bram Moolenaar wrote: > > >> getchar.c: In function 'check_map': > >> getchar.c:5221:7: warning: assuming signed overflow does not occur when > >> assuming that (X - c) <= X is always true [-Wstrict-overflow] > >> if (len > mp->m_keylen - 3) > >> ^ > > > >

Re: Why "loadplugin" when there's already "loadplugins"?

2016-02-22 Fir de Conversatie Charles Campbell
Bram Moolenaar wrote: > Right, the name is not ideal, but I could not think of a better one. I > would also use "bundle" instead of package or pack, but plugin > managers already use it and there could be a conflict. I hope that the > new mechanism doesn't break existing behavior. Nikolay, you had

[PATCH] Compiler warning in ex_cmds2.c [after 7.4.1384]

2016-02-22 Fir de Conversatie tux.
Well, well. -- -- 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" gro

Re: Patch 7.4.1375

2016-02-22 Fir de Conversatie Bram Moolenaar
Kenichi Ito wrote: > The help link still remains in the help toc. Thanks! -- ARTHUR: Shut up! Will you shut up! DENNIS: Ah, now we see the violence inherent in the system. ARTHUR: Shut up! DENNIS: Oh! Come and see the violence inherent in the system! HELP! HELP! I'm being repr

Patch 7.4.1387

2016-02-22 Fir de Conversatie Bram Moolenaar
Patch 7.4.1387 Problem:Win16 docs still referenced. Solution: Remove Win16 files from the docs Makefile. (Kenichi Ito) Files: runtime/doc/Makefile *** ../vim-7.4.1386/runtime/doc/Makefile2016-01-28 22:36:15.056065002 +0100 --- runtime/doc/Makefile2016-02-22 20:04:06.9

Re: Patch 7.4.1375

2016-02-22 Fir de Conversatie Bram Moolenaar
Kenichi Ito wrote: > I found forgetting to remove the statements about gui_w16.txt from Makefile. > The patch is here. Thanks! -- Seen on the back of a biker's vest: If you can read this, my wife fell off. /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net \\\ ///

Re: Why "loadplugin" when there's already "loadplugins"?

2016-02-22 Fir de Conversatie Nikolay Aleksandrovich Pavlov
2016-02-22 12:05 GMT+03:00 Bram Moolenaar : > > Nikolay Pavlov wrote: > >> 2016-02-22 5:08 GMT+03:00 Manuel Ortega : >> > I know that loadplugins in an option while loadplugin is a command, but >> > surely there is a better name for the new command. This is just >> > gratuitously confusing. >> > >

Re: Warnings in ex_cmds2.c, getchar.c, pathdef.c, undo.c, os_win32.c

2016-02-22 Fir de Conversatie Dominique Pellé
"Bram Moolenaar wrote: >> getchar.c: In function 'check_map': >> getchar.c:5221:7: warning: assuming signed overflow does not occur when >> assuming that (X - c) <= X is always true [-Wstrict-overflow] >> if (len > mp->m_keylen - 3) >> ^ > > That looks like a compiler problem. Line

Re: job_start('echo hi') hangs on Linux, GTK2 GUI

2016-02-22 Fir de Conversatie mattn
Bram, it seems no one close the channel. diff --git a/src/channel.c b/src/channel.c index aa97088..0cf8718 100644 --- a/src/channel.c +++ b/src/channel.c @@ -1708,7 +1708,7 @@ channel_read(channel_T *channel, int part, char *func) /* Reading a socket disconnection (readlen == 0), or a socke

Re: Warnings in ex_cmds2.c, getchar.c, pathdef.c, undo.c, os_win32.c

2016-02-22 Fir de Conversatie Bram Moolenaar
Cesar Romani wrote: > By building vim 7.4.1385 on Windows 7 with MinGW, I get following > warnings: > > > [...] > ex_cmds2.c: In function 'source_pack_plugin': > ex_cmds2.c:3076:9: warning: 'p6' may be used uninitialized in this > function [-Wmaybe-uninitialized] > c

job_start('echo hi') hangs on Linux, GTK2 GUI

2016-02-22 Fir de Conversatie tyru
Hi Vimmers! I have compiled Vim 7.4.1386 with GTK2 GUI. And tested the following command: echo job_start('echo hi') It hangs only in GTK2 GUI. CUI Vim does not hang, though. My :version output is here. https://gist.github.com/tyru/9647351555282ebc2509 -- Takuya Fujiwara -- -- You receiv

Highlight Vim folds depending on their levels

2016-02-22 Fir de Conversatie Laurent Humblet
Hi, I was looking to make my inner folds more visible when they are beside the normal folds. I wanted to use highlights for this but it seems not possible atm. Herbert Sitz seems to have found a solution for this and his patches are in his GitHub folder but they don't seem to work out-of-the-box

Re: Patch 7.4.1375

2016-02-22 Fir de Conversatie h_east
Hi Kenichi, 2016-2-22(Mon) 23:50:43 UTC+9 Kenichi Ito: > Hi, > > I found forgetting to remove the statements about gui_w16.txt from Makefile. > The patch is here. > > diff --git a/runtime/doc/Makefile b/runtime/doc/Makefile > index 6bbcbc3..d36f2ce 100644 > --- a/runtime/doc/Makefile > +++ b/run

Re: [patch] Add Error autocmd event

2016-02-22 Fir de Conversatie Anton Lindqvist
Realized I was using my personal preference for cinoptions. Here's a revised patch with correct indentation. -- :wq -- -- 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/m

Re: Patch 7.4.1375

2016-02-22 Fir de Conversatie Kenichi Ito
Hi, I found forgetting to remove the statements about gui_w16.txt from Makefile. The patch is here. diff --git a/runtime/doc/Makefile b/runtime/doc/Makefile index 6bbcbc3..d36f2ce 100644 --- a/runtime/doc/Makefile +++ b/runtime/doc/Makefile @@ -32,7 +32,6 @@ DOCS = \ ft_ada.txt \

Re: test_channel.vim fails with Win32 CUI Vim

2016-02-22 Fir de Conversatie Yukihiro Nakadaira
On Mon, Feb 22, 2016 at 9:27 PM, Bram Moolenaar wrote: > > Yukihiro Nakadaira wrote: > > > > > test_channel.vim fails with Win32 CUI Vim. > > > > > > > > What the following patch fix is > > > > 1. Vim sleep 5 msec even when timeout=0. > > > > 2. channel_handle_events() closes channel when there i

Re: Compiler warning

2016-02-22 Fir de Conversatie Ken Takata
Hi, 2016/2/22 Mon 22:32:17 UTC+9 Tony Mechelynck wrote: > After updating my Vim clone to v7.4.1386, the same message appears in > both Huge and Tiny: > > gcc -c -I. -Iproto -DHAVE_CONFIG_H -O2 -fno-strength-reduce -Wall > -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -o objects/ex_cmds2.o > ex_

Re: Patch 7.4.1375

2016-02-22 Fir de Conversatie Kenichi Ito
Hi, The help link still remains in the help toc. --- diff --git a/runtime/doc/help.txt b/runtime/doc/help.txt index 23ad1dc..159135f 100644 --- a/runtime/doc/help.txt +++ b/runtime/doc/help.txt @@ -155,7 +155,6 @@ Special issues ~ GUI ~ |gui.txt| Graphical User Interface (GUI) -|gui_w16.

Re: Setting autochdir AND encoding=utf-8 has gvim -d fail

2016-02-22 Fir de Conversatie Christian Brabandt
Hi Axel! On So, 21 Feb 2016, Axel Bender wrote: > Yup, at 1379 here. Yes, I see the problem. It's caused by having :set acd change to the directory of the first buffer when initializing the options. Later we call fix_arg_enc() which tries to guess the filenames again, butt since 'autochdir' h

Compiler warning

2016-02-22 Fir de Conversatie Tony Mechelynck
After updating my Vim clone to v7.4.1386, the same message appears in both Huge and Tiny: gcc -c -I. -Iproto -DHAVE_CONFIG_H -O2 -fno-strength-reduce -Wall -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -o objects/ex_cmds2.o ex_cmds2.c ex_cmds2.c: In function ‘source_pack_plugin’: ex_cmds2.c:3076

Re: test_channel.vim fails with Win32 CUI Vim

2016-02-22 Fir de Conversatie Bram Moolenaar
Yukihiro Nakadaira wrote: > > > test_channel.vim fails with Win32 CUI Vim. > > > > > > What the following patch fix is > > > 1. Vim sleep 5 msec even when timeout=0. > > > 2. channel_handle_events() closes channel when there is no input. > > > 3. ":sleep" command does not read channel (I am not s

Re: test_channel.vim fails with Win32 CUI Vim

2016-02-22 Fir de Conversatie Yukihiro Nakadaira
On Mon, Feb 22, 2016 at 1:19 AM, Bram Moolenaar wrote: > > Yukihiro Nakadaira wrote: > > > test_channel.vim fails with Win32 CUI Vim. > > > > What the following patch fix is > > 1. Vim sleep 5 msec even when timeout=0. > > 2. channel_handle_events() closes channel when there is no input. > > 3. "

Re: Very big file gets truncated

2016-02-22 Fir de Conversatie Andreas
On Monday, 22 February 2016 12:21:12 UTC+1, Ken Takata wrote: > Hi Andreas, > > 2016/2/22 Mon 19:44:25 UTC+9 Ken Takata wrote: > > Thank you for testing my patches. > > Did you apply 0002_fix-off_T-display.patch? Without this, character count > > will not be shown properly when a file has been r

Re: Very big file gets truncated

2016-02-22 Fir de Conversatie Ken Takata
Hi Andreas, 2016/2/22 Mon 19:44:25 UTC+9 Ken Takata wrote: > Thank you for testing my patches. > Did you apply 0002_fix-off_T-display.patch? Without this, character count > will not be shown properly when a file has been read/written. > Or did you check the character count in another way? Ah oka

Re: Very big file gets truncated

2016-02-22 Fir de Conversatie Ken Takata
Hi Andreas, 2016/2/22 Mon 18:05:35 UTC+9 Andreas wrote: > On Sunday, 21 February 2016 16:20:48 UTC+1, Andreas wrote: > > On Friday, 19 February 2016 13:00:39 UTC+1, Ken Takata wrote: > > > Hi Andreas, > > > > > > 2016/2/19 Fri 20:47:20 UTC+9 anst...@gmail.com wrote: > > > > I have a text file (

Patch 7.4.1386

2016-02-22 Fir de Conversatie Bram Moolenaar
Patch 7.4.1386 Problem:When the Job exit callback is invoked, the job may be freed too soon. (Yasuhiro Matsumoto) Solution: Increase refcount. Files: src/eval.c *** ../vim-7.4.1385/src/eval.c 2016-02-21 23:12:36.546500335 +0100 --- src/eval.c 2016-02-22 11:38:27.56065612

Re: Patch 7.4.1380

2016-02-22 Fir de Conversatie Bram Moolenaar
Yasuhiro Matsumoto wrote: > On Monday, February 22, 2016 at 2:29:12 PM UTC+9, mattn wrote: > > On Monday, February 22, 2016 at 2:23:49 PM UTC+9, mattn wrote: > > > Possibly double free. And small refactoring. > > > > > > https://gist.github.com/mattn/cc99c7a39598c5c632c9 > > > > Sorry, my fault

Re: Very big file gets truncated

2016-02-22 Fir de Conversatie Andreas
On Sunday, 21 February 2016 16:20:48 UTC+1, Andreas wrote: > On Friday, 19 February 2016 13:00:39 UTC+1, Ken Takata wrote: > > Hi Andreas, > > > > 2016/2/19 Fri 20:47:20 UTC+9 anst...@gmail.com wrote: > > > I have a text file (dc.epd) that is nearly 8GB. When I open it in > > > gvim.exe version

Re: Why "loadplugin" when there's already "loadplugins"?

2016-02-22 Fir de Conversatie Bram Moolenaar
Nikolay Pavlov wrote: > 2016-02-22 5:08 GMT+03:00 Manuel Ortega : > > I know that loadplugins in an option while loadplugin is a command, but > > surely there is a better name for the new command. This is just > > gratuitously confusing. > > > > What's wrong with "loadpack" or something like tha

Re: [PATCH] Convert formatprg to a global-local option.

2016-02-22 Fir de Conversatie Justin M. Keyes
On Wednesday, April 24, 2013 at 8:38:42 PM UTC-4, Gary Johnson wrote: > On 2013-04-24, Sung Pae wrote: > > > So I retract my last response: a global-local formatprg is clearly > > useful for both prose and code, and comes at the cost of a small > > non-breaking change. > > > > Tony Mechelynck's c