Compilation warning in os_win32.c

2016-02-23 Fir de Conversatie Axel Bender
Windows 7 64-bit, MinGw 64 >>>CONSOLE<<< gcc -c -Iproto -DWIN32 -DWINVER=0x0501 -D_WIN32_WINNT=0x0501 -DHAVE_PATHDEF -DFEAT_BIG -DMS_WIN64 -DHAVE_GETTEXT -DHAVE_LOCALE_H -DDYNAMIC_GETTEXT -DFEAT_CSCOPE -DFEAT_CHANNEL -DFEAT_MBYTE -DFEAT_MBYTE_IME -DDYNAMIC_IME -DDYNAMIC_ICONV -pipe

Compilation warning in undo.c

2016-02-23 Fir de Conversatie Axel Bender
Windows 7 64-bit, MinGw 64 >>>GUI<<< gcc -c -Iproto -DWIN32 -DWINVER=0x0501 -D_WIN32_WINNT=0x0501 -DHAVE_PATHDEF -DFEAT_BIG -DMS_WIN64 -DHAVE_GETTEXT -DHAVE_LOCALE_H -DDYNAMIC_GETTEXT -DFEAT_CSCOPE -DFEAT_CHANNEL -DFEAT_GUI_W32 -DFEAT_CLIPBOARD -DFEAT_MBYTE -DFEAT_MBYTE_IME -DDYNAMIC_IME

Compilation warning in getchar.c

2016-02-23 Fir de Conversatie Axel Bender
Windows 7 64-bit, MinGw 64 >>>CONSOLE<<< gcc -c -Iproto -DWIN32 -DWINVER=0x0501 -D_WIN32_WINNT=0x0501 -DHAVE_PATHDEF -DFEAT_BIG -DMS_WIN64 -DHAVE_GETTEXT -DHAVE_LOCALE_H -DDYNAMIC_GETTEXT -DFEAT_CSCOPE -DFEAT_CHANNEL -DFEAT_MBYTE -DFEAT_MBYTE_IME -DDYNAMIC_IME -DDYNAMIC_ICONV -pipe

close-cb is called even when closed by ch_close()

2016-02-23 Fir de Conversatie Christian J. Robinson
The documentation says: "close-cb"A function that is called when the channel gets closed, other than by calling ch_close(). However, I still get the close callback triggered when I use ch_close(). - Christian -- If you explain so that nobody misunderstands,

compilation warnings with gcc-6 in farsi.c

2016-02-23 Fir de Conversatie Dominique Pellé
Hi I compiled vim-7.4.1411 with gcc-6 (a snapshot, as it's not released yet). gcc-6 gave warnings in farsi.c: farsi.c: In function ‘fkmap’: farsi.c:767:7: warning: statement is indented as if it were guarded by... [-Wmisleading-indentation] chg_c_toX_orX (); ^

Re: Patch 7.4.1408

2016-02-23 Fir de Conversatie Nikolay Aleksandrovich Pavlov
2016-02-24 0:08 GMT+03:00 Bram Moolenaar : > > Patch 7.4.1408 > Problem:MS-Windows doesn't have isnan() and isinf(). > Solution: Use _isnan() and _isinf(). > Files: src/eval.c, src/json.c Wondering why [_]isnan and not fpclassify? > > > ***

Re: [vim] json_encode incorrectly accepts and encodes "inf" and "nan" (#654)

2016-02-23 Fir de Conversatie Nikolay Aleksandrovich Pavlov
2016-02-23 23:27 GMT+03:00 Bram Moolenaar : > > David Barnett wrote: > > > `json_encode` accepts special "inf" and "nan" float values and renders > them literally into the encoded JSON: > > ```vim > > :echo json_encode(0.0 / 0.0) > > '-nan' > > ``` > > > > Those values

Patch 7.4.1411

2016-02-23 Fir de Conversatie Bram Moolenaar
Patch 7.4.1411 Problem:Compiler warning for indent. (Ajit Thakkar) Solution: Indent normally. Files: src/ui.c *** ../vim-7.4.1410/src/ui.c2016-01-31 14:55:35.231538430 +0100 --- src/ui.c2016-02-23 23:00:20.619567100 +0100 *** *** 711,717 if (row1 <

Re: [bug] Compiling vim.exe fails

2016-02-23 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > On Di, 23 Feb 2016, Bram Moolenaar wrote: > > > The output of :version should tell you if there is a fall-back for > > $VIMRUNTIME. If it's not set, then check the Makefile, in Make_mvc.mak > > it's done around line 1323. However, the default is empty. > > > > It

Re: Patch 7.4.1406

2016-02-23 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > On Di, 23 Feb 2016, Bram Moolenaar wrote: > > > > > Patch 7.4.1406 > > Problem:Leaking memory in cs_print_tags_priv(). > > Solution: Free tbuf. (idea by Forrest Fleming) > > Files: src/if_cscope.c > > I think, we also need this: > > index

Patch 7.4.1410

2016-02-23 Fir de Conversatie Bram Moolenaar
Patch 7.4.1410 Problem:Leaking memory in cscope interface. Solution: Free memory when no tab is found. (Christian Brabandt) Files: src/if_cscope.c *** ../vim-7.4.1409/src/if_cscope.c 2016-02-23 20:43:58.725419739 +0100 --- src/if_cscope.c 2016-02-23 22:47:02.879951592 +0100

Re: [bug] Compiling vim.exe fails

2016-02-23 Fir de Conversatie Christian Brabandt
On Di, 23 Feb 2016, Bram Moolenaar wrote: > The output of :version should tell you if there is a fall-back for > $VIMRUNTIME. If it's not set, then check the Makefile, in Make_mvc.mak > it's done around line 1323. However, the default is empty. > > It should work to find the runtime files

Patch 7.4.1409

2016-02-23 Fir de Conversatie Bram Moolenaar
Patch 7.4.1409 (after 7.4.1402) Problem:Configure includes GUI despite --disable-gui flag. Solution: Add SKIP_GTK3. (Kazunobu Kuriyama) Files: src/configure.in, src/auto/configure *** ../vim-7.4.1408/src/configure.in2016-02-23 17:13:56.873032372 +0100 --- src/configure.in

Patch 7.4.1408

2016-02-23 Fir de Conversatie Bram Moolenaar
Patch 7.4.1408 Problem:MS-Windows doesn't have isnan() and isinf(). Solution: Use _isnan() and _isinf(). Files: src/eval.c, src/json.c *** ../vim-7.4.1407/src/eval.c 2016-02-23 21:26:38.770552198 +0100 --- src/eval.c 2016-02-23 21:47:32.765412226 +0100 *** *** 27,34

Patch 7.4.1407

2016-02-23 Fir de Conversatie Bram Moolenaar
Patch 7.4.1407 Problem:json_encode() does not handle NaN and inf properly. (David Barnett) Solution: For JSON turn them into "null". For JS use "NaN" and "Infinity". Add isnan(). Files: src/eval.c, src/json.c, src/testdir/test_json.vim ***

Re: Patch 7.4.1406

2016-02-23 Fir de Conversatie Christian Brabandt
Hi Bram! On Di, 23 Feb 2016, Bram Moolenaar wrote: > > Patch 7.4.1406 > Problem:Leaking memory in cs_print_tags_priv(). > Solution: Free tbuf. (idea by Forrest Fleming) > Files: src/if_cscope.c I think, we also need this: index b5ca614..2f75390 100644 --- a/src/if_cscope.c +++

Re: [patch] neovim patches

2016-02-23 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > another one: > https://patch-diff.githubusercontent.com/raw/neovim/neovim/pull/4336.patch Thanks. Can be made shorter... -- Q: What's orange and sounds like a parrot? A: A carrot /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net \\\ ///

Patch 7.4.1406

2016-02-23 Fir de Conversatie Bram Moolenaar
Patch 7.4.1406 Problem:Leaking memory in cs_print_tags_priv(). Solution: Free tbuf. (idea by Forrest Fleming) Files: src/if_cscope.c *** ../vim-7.4.1405/src/if_cscope.c 2016-02-16 15:06:54.661635316 +0100 --- src/if_cscope.c 2016-02-23 20:41:15.027135552 +0100 ***

Re: Patch 7.4.1398

2016-02-23 Fir de Conversatie Bram Moolenaar
Mats Bertil Tegner wrote: > On 2016-02-23 13:20, Bram Moolenaar wrote: > > > > Patch 7.4.1398 > > Problem:The close-cb option is not implemented yet. > > Solution: Implemente close-cb. (Yasuhiro Matsumoto) > > Files: src/channel.c, src/eval.c, src/structs.h, src/proto/channel.pro, >

Patch 7.4.1405

2016-02-23 Fir de Conversatie Bram Moolenaar
Patch 7.4.1405 Problem:Completion menu flickers. Solution: Delay showing the popup menu. (Shougo, Justin M. Keyes, closes #656) Files: src/edit.c *** ../vim-7.4.1404/src/edit.c 2016-02-16 20:09:07.865342593 +0100 --- src/edit.c 2016-02-23 20:10:43.558344423 +0100

Re: [patch] neovim patches

2016-02-23 Fir de Conversatie Christian Brabandt
another one: https://patch-diff.githubusercontent.com/raw/neovim/neovim/pull/4336.patch Best, Christian -- Steht im Winter noch das Korn, ist es wohl vergessen wor'n. -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying

Re: When GUI disabled, configure tries to enable GTK3 (and compiling gui_gtk_x11.c fails)

2016-02-23 Fir de Conversatie Kazunobu Kuriyama
Hi Tony, 2016-02-24 3:39 GMT+09:00 Tony Mechelynck : > N.B. In my Huge build, which has --enable-gnome-check but neither > --enable nor --disable -gui, configure sets gui=gtk2 as expected. The > following errors are in my Tiny build, which has --disable-gui > >

When GUI disabled, configure tries to enable GTK3 (and compiling gui_gtk_x11.c fails)

2016-02-23 Fir de Conversatie Tony Mechelynck
N.B. In my Huge build, which has --enable-gnome-check but neither --enable nor --disable -gui, configure sets gui=gtk2 as expected. The following errors are in my Tiny build, which has --disable-gui IMHO, when --disable-gui, configure should skip the GTK3 test and enable no GUI of any kind. The

Re: problems with building vim from patches

2016-02-23 Fir de Conversatie Bram Moolenaar
Charles Campbell wrote: > I don't build vim as often as I used to since patch#866, which hangs vim > when used with the server-client calls. > > I started fresh with vim74.tar and attempted to apply all patches from 1 > to 1402 (and they were applied in the proper sequence). I'm ignoring > any

Patch 7.4.1404

2016-02-23 Fir de Conversatie Bram Moolenaar
Patch 7.4.1404 Problem:ch_read() doesn't time out on MS-Windows. Solution: Instead of WM_NETBEANS use select(). (Yukihiro Nakadaira) Files: src/channel.c, src/gui_w32.c, src/os_win32.c, src/structs.h, src/testdir/test_channel.vim, src/vim.h ***

Re: Solaris needs a tiny waittime, too

2016-02-23 Fir de Conversatie Bram Moolenaar
Danek Duvall wrote: > 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 > > >

Re: test_channel.vim fails with Win32 CUI Vim

2016-02-23 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

Re: Patch 7.4.1398

2016-02-23 Fir de Conversatie Mats Bertil Tegner
On 2016-02-23 13:20, Bram Moolenaar wrote: > > Patch 7.4.1398 > Problem:The close-cb option is not implemented yet. > Solution: Implemente close-cb. (Yasuhiro Matsumoto) > Files: src/channel.c, src/eval.c, src/structs.h, src/proto/channel.pro, > src/testdir/test_channel.py,

Patch - Not able to build Vim without the quickfix feature

2016-02-23 Fir de Conversatie Yegappan Lakshmanan
Hi, The attached patch fixes the errors seen when building Vim without the quickfix feature. - 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

Patch 7.4.1401

2016-02-23 Fir de Conversatie Bram Moolenaar
Patch 7.4.1401 Problem:Having 'autochdir' set during startup and using diff mode doesn't work. (Axel Bender) Solution: Don't use 'autochdir' while still starting up. (Christian Brabandt) Files: src/buffer.c *** ../vim-7.4.1400/src/buffer.c2016-02-23

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

2016-02-23 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > 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

Re: perleval bug on 64-bit big-endian architectures

2016-02-23 Fir de Conversatie Bram Moolenaar
Danek Duvall wrote: > 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.

Re: [vim] Netrw: special directory names causes Vim to display directories as if they were empty [Bug] (#657)

2016-02-23 Fir de Conversatie Charles Campbell
Jelly wrote: > > I discovered this whilst trying to use Vim's Netrw as my file manager > (for more generic use). > Me, including two other experienced users have successfully reproduced > what seems to be a bug. > > *Reproducing the bug:* > > |$ mkdir "Foo [Bar]" && cd $_ $ touch a b c $ vim . | >

Re: [vim] sh.vim incorrectly highlighting if/fi (#636)

2016-02-23 Fir de Conversatie Charles Campbell
mattn wrote: > > try following on cursor at |--install|. > > |echo synIDattr(synID(line("."), col("."), 1), "name") | > > what do you get? > > Using hilinks.vim, which gives rather more information, and with the cursor on --install: SynStack: shOption

Patch 7.4.1398

2016-02-23 Fir de Conversatie Bram Moolenaar
Patch 7.4.1398 Problem:The close-cb option is not implemented yet. Solution: Implemente close-cb. (Yasuhiro Matsumoto) Files: src/channel.c, src/eval.c, src/structs.h, src/proto/channel.pro, src/testdir/test_channel.py, src/testdir/test_channel.vim ***

Re: [bug] Compiling vim.exe fails

2016-02-23 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 >

Re: implementation of close-cb

2016-02-23 Fir de Conversatie Bram Moolenaar
Yasuhiro Matsumoto wrote: > Hi, list. > > one of TODO items. > > https://gist.github.com/mattn/443f603e3c17f3de3391 Thanks! I think we need to keep the channel around even when the refcount goes down to zero, if the callback would still be invoked. -- GUEST:He's killed the best

Re: [vim] Support defining compilation date in $SOURCE_DATE_EPOCH (#513)

2016-02-23 Fir de Conversatie Tony Mechelynck
On Thu, Dec 3, 2015 at 2:22 AM, James McCoy wrote: > > There is an ongoing effort0 to make FOSS software reproducibly > buildable. In order to make Vim build reproducibly, it is necessary to > allow defining the date/time that is part of VIM_VERSION_LONG as part of >

implementation of close-cb

2016-02-23 Fir de Conversatie mattn
Hi, list. one of TODO items. https://gist.github.com/mattn/443f603e3c17f3de3391 - 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

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

2016-02-23 Fir de Conversatie Christ van Willegen
On Mon, Feb 22, 2016 at 11:13 PM, Bram Moolenaar wrote: > > Tux wrote: > >> Well, well. > > Sorry, I find it difficult to count beyond 4. :-) I find that difficult to believe since VI is 6 in roman numerals ;-) Christ van Willegen -- -- You received this message from the

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

2016-02-23 Fir de Conversatie Dominique Pellé
Bram Moolenaar wrote: > 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] >> >>