Re: Patch 7.4.1380

2016-02-21 Fir de Conversatie mattn
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. > > So this is only refactoring. > > h

Re: Patch 7.4.1380

2016-02-21 Fir de Conversatie mattn
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. So this is only refactoring. https://gist.github.com/mattn/6f298ada2eee242c178c -- -- You received this messag

Re: Patch 7.4.1380

2016-02-21 Fir de Conversatie mattn
Possibly double free. And small refactoring. https://gist.github.com/mattn/cc99c7a39598c5c632c9 -- -- 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

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

2016-02-21 Fir de Conversatie Jacob Niehus
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 sort() is called while already

Fwd: [Patch] GTK3 GUI

2016-02-21 Fir de Conversatie Kazunobu Kuriyama
Oops, I failed to send to the list. Sorry. Kazunobu -- Forwarded message -- From: Kazunobu Kuriyama Date: 2016-02-22 13:03 GMT+09:00 Subject: Re: [Patch] GTK3 GUI To: Bram Moolenaar Patch updated, made against v7.4.1385. As I wrote last Saturday, this is a version I reviewed

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

2016-02-21 Fir de Conversatie Nikolay Aleksandrovich Pavlov
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 that? I would say that `:loadpack` nam

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

2016-02-21 Fir de Conversatie Cesar Romani
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 = *p6; ^ [...] getchar.c: In

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

2016-02-21 Fir de Conversatie 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 that? -Manny -- -- You received this message from the "vim_dev" maillist. Do not top-p

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

2016-02-21 Fir de Conversatie Axel Bender
Yup, at 1379 here. -- -- 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_d

Re: Patch to fix warning in eval.c

2016-02-21 Fir de Conversatie Bram Moolenaar
Yegappan Lakshmanan wrote: > The attached patch fixes a warning message seen when building the > latest version of Vim. Thanks! Wrong autocomplete... -- Individualists unite! /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net \\\ ///sponsor Vim, vote for feature

Patch 7.4.1385

2016-02-21 Fir de Conversatie Bram Moolenaar
Patch 7.4.1385 Problem:Compiler warning for using array. Solution: Use the right member name. (Yegappan Lakshmanan) Files: src/eval.c *** ../vim-7.4.1384/src/eval.c 2016-02-21 23:01:47.449323306 +0100 --- src/eval.c 2016-02-21 23:10:04.804095172 +0100 *** *** 10135,10141

Patch 7.4.1384

2016-02-21 Fir de Conversatie 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'. Files: src/main.c, src/ex_cmds2.c, src/option.c, src/option.h, src/ex_cmds.h, src/eval.c, src/version.c, src/proto/ex_cmds2.pro,

Patch to fix warning in eval.c

2016-02-21 Fir de Conversatie Yegappan Lakshmanan
Hi, The attached patch fixes a warning message seen when building the latest version of Vim. - Yegappan -- -- 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

Re: [bug] Compiling vim.exe fails

2016-02-21 Fir de Conversatie Christian Brabandt
Hi Bram! On Fr, 19 Feb 2016, Bram Moolenaar wrote: > > 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. Si

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

2016-02-21 Fir de Conversatie Christian Brabandt
Hi Axel! On So, 21 Feb 2016, Axel Bender wrote: > Windows 7 64-bit, with MinGW 64 > > Being in z:\lib\vim, and d:\temp\.vimrc only containing: > > > set autochdir > set encoding=utf-8 > > > gvim -d x86\make_cyg_ming.mak x64\make_cyg_ming.mak -u d:\temp\.vimrc > (sam

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

2016-02-21 Fir de Conversatie Axel Bender
Windows 7 64-bit, with MinGW 64 Being in z:\lib\vim, and d:\temp\.vimrc only containing: set autochdir set encoding=utf-8 gvim -d x86\make_cyg_ming.mak x64\make_cyg_ming.mak -u d:\temp\.vimrc (sample) fails, as gvim doesn't find the first file. The name of the first b

Patch 7.4.1383

2016-02-21 Fir de Conversatie Bram Moolenaar
Patch 7.4.1383 Problem:GvimExt only loads the old libintl.dll. Solution: Also try loading libint-8.dll. (Ken Takata, closes #608) Files: src/GvimExt/gvimext.cpp, src/GvimExt/gvimext.h *** ../vim-7.4.1382/src/GvimExt/gvimext.cpp 2015-05-04 18:27:29.920714802 +0200 --- src/GvimExt/

Patch 7.4.1382

2016-02-21 Fir de Conversatie Bram Moolenaar
Patch 7.4.1382 Problem:Can't get the job of a channel. Solution: Add ch_getjob(). Files: src/eval.c, runtime/doc/channel.txt, runtime/doc/eval.txt *** ../vim-7.4.1381/src/eval.c 2016-02-21 19:14:36.675958737 +0100 --- src/eval.c 2016-02-21 19:47:07.503682158 +0100 *** **

Patch 7.4.1381

2016-02-21 Fir de Conversatie Bram Moolenaar
Patch 7.4.1381 (after 7.4.1380) Problem:Exit value not available on MS-Windows. Solution: Set the exit value. Files: src/structs.h, src/os_win32.c *** ../vim-7.4.1380/src/structs.h 2016-02-21 19:14:36.671958779 +0100 --- src/structs.h 2016-02-21 19:24:26.821822302 +0100 **

Patch 7.4.1380

2016-02-21 Fir de Conversatie Bram Moolenaar
Patch 7.4.1380 Problem:The job exit callback is not implemented. Solution: Add the "exit-cb" option. Files: src/structs.h, src/eval.c, src/channel.c, src/proto/eval.pro, src/misc2.c, src/macros.h, src/testdir/test_channel.vim *** ../vim-7.4.1379/src/structs.h 2016-02

Patch 7.4.1379

2016-02-21 Fir de Conversatie Bram Moolenaar
Patch 7.4.1379 Problem:Channel test fails on Win32 console. Solution: Don't sleep when timeout is zero. Call channel_wait() before channel_read(). Channels are not polled during ":sleep". (Yukihiro Nakadaira) Files: src/channel.c, src/misc2.c, src/gui_w32.c, src/

Re: test_channel.vim fails with Win32 CUI Vim

2016-02-21 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 sure where is the right

Re: ch_sendexpr and a callback

2016-02-21 Fir de Conversatie Bram Moolenaar
Marcin Szamotulski wrote: > The ch_sendexpr requires a callback to run asynchronously. In my > scenario I add a listener od the channel and still I'd like to run > ch_sendexpr asynchronous, I can add the same callback as on the > channel, but this seems not intuitive. I'd suggest to use a boole

Re: Patch 7.4.1368

2016-02-21 Fir de Conversatie h_east
Hi Bram and List, 2016-2-21(Sun) 6:16:04 UTC+9 Bram Moolenaar: > Hirohito Higashi wrote: > > > Hi Bram and list, > > > > 2016-2-20(Sat) 23:51:14 UTC+9 Bram Moolenaar: > > > Patch 7.4.1368 > > > Problem:One more Win16 file remains. > > > Solution: Delete it. > > > Files: src/proto/os_w

Patch 7.4.1378

2016-02-21 Fir de Conversatie Bram Moolenaar
Patch 7.4.1378 Problem:Can't change job settings after it started. Solution: Add job_setoptions() with the "stoponexit" flag. Files: src/eval.c, src/main.c, src/structs.h, src/proto/eval.pro, src/testdir/test_channel.vim *** ../vim-7.4.1377/src/eval.c 2016-02-20 23:30:02.

Re: Very big file gets truncated

2016-02-21 Fir de Conversatie anstabel
On Friday, 19 February 2016 13:35:02 UTC+1, Ken Takata wrote: > Hi Tony, > > 2016/2/19 Fri 21:01:35 UTC+9 Tony Mechelynck wrote: > > On Fri, Feb 19, 2016 at 11:35 AM, wrote: > > > I have a text file (dc.epd) that is nearly 8GB. When I open it in > > > gvim.exe version 1353 the display shows "d

Re: Very big file gets truncated

2016-02-21 Fir de Conversatie anstabel
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 1353 the display shows "dc.epd" 0 lines, 0 characters and the file > >

test_channel.vim fails with Win32 CUI Vim

2016-02-21 Fir de Conversatie Yukihiro Nakadaira
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 sure where is the right place to call channel_handle_events()

Re: [patch] Add optional argument to bufwinnr to return all windows

2016-02-21 Fir de Conversatie Anton Lindqvist
On Sun, Feb 21, 2016 at 03:15:17PM +0100, Bram Moolenaar wrote: > > Anton Lindqvist wrote: > > > Hi, > > This patch adds an optional second argument to the bufwinnr function > > which if present returns all windows associated with the given buffer > > across all tabs. This is useful in order to d

ch_sendexpr and a callback

2016-02-21 Fir de Conversatie mszamot
Hi Bram, The ch_sendexpr requires a callback to run asynchronously. In my scenario I add a listener od the channel and still I'd like to run ch_sendexpr asynchronous, I can add the same callback as on the channel, but this seems not intuitive. I'd suggest to use a boolean option passed to ch_sen

Re: [patch] fixed typo in runtime/doc/channel.txt

2016-02-21 Fir de Conversatie Bram Moolenaar
Dominique wrote: > Attached patch fixes a typo in runtime/doc/channel.txt. Thanks. I had already fixed it when changing "eval" to "expr". -- A mathematician is a device for turning coffee into theorems. Paul Erdos A computer programmer is a device for

Re: [patch] Add optional argument to bufwinnr to return all windows

2016-02-21 Fir de Conversatie Bram Moolenaar
Anton Lindqvist wrote: > Hi, > This patch adds an optional second argument to the bufwinnr function > which if present returns all windows associated with the given buffer > across all tabs. This is useful in order to determine if a buffer is > present in several windows. I'm currently using this

Re: Bug: Windows, ch_open() returns a channel handle even on failure

2016-02-21 Fir de Conversatie Bram Moolenaar
Christian J. Robinson wrote: > On Windows 7, without a server to connect to, this will echo a "9" > every time: > >:let open_config = {'mode': 'json', 'timeout': 2000, 'waittime': 2000} >:let handle = ch_open('localhost:9876', open_config) >:echomsg type(handle) Yes. Perhaps you w

Patch 7.4.1377

2016-02-21 Fir de Conversatie Bram Moolenaar
Patch 7.4.1377 Problem:Test_connect_waittime() is flaky. Solution: Ignore the "Connection reset by peer" error. Files: src/testdir/test_channel.vim *** ../vim-7.4.1376/src/testdir/test_channel.vim2016-02-20 23:30:02.884652677 +0100 --- src/testdir/test_channel.vim2016

[patch] Add optional argument to bufwinnr to return all windows

2016-02-21 Fir de Conversatie Anton Lindqvist
Hi, This patch adds an optional second argument to the bufwinnr function which if present returns all windows associated with the given buffer across all tabs. This is useful in order to determine if a buffer is present in several windows. I'm currently using this patch in order to get backspace to