Olaf Dabrunz | 19 Aug 21:09 2014

[Patch] Fix error on empty input for system()

Hello,

to reproduce this in vim:

    :echo system('md5sum', '')
    E677: Error writing temp file

With the patch below, there is no error and the result is as expected:

    :echo system('md5sum', '')
    d41d8cd98f00b204e9800998ecf8427e  -

Tested against vim 7.4.000 on Ubuntu 12.04. The patch below is against
the current hg repo.

diff -r 7af530381fec src/eval.c
--- a/src/eval.c	Sun Aug 17 17:24:07 2014 +0200
+++ b/src/eval.c	Tue Aug 19 21:13:14 2014 +0200
 <at>  <at>  -18575,13 +18575,16  <at>  <at> 
 	}
 	else
 	{
+	    size_t len;
+
 	    p = get_tv_string_buf_chk(&argvars[1], buf);
 	    if (p == NULL)
 	    {
 		fclose(fd);
 		goto errret;		/* type error; errmsg already given */
 	    }
(Continue reading)

Yukihiro Nakadaira | 19 Aug 14:34 2014
Picon

doc fix for :help arglistid()

According to :help arglistid(), it returns zero for invalid argument.
But it returns -1 instead.

diff -r 7af530381fec runtime/doc/eval.txt
--- a/runtime/doc/eval.txt    Sun Aug 17 17:24:07 2014 +0200
+++ b/runtime/doc/eval.txt    Tue Aug 19 21:28:25 2014 +0900
<at> <at> -2113,7 +2113,7 <at> <at>
         Return the argument list ID.  This is a number which
         identifies the argument list being used.  Zero is used for the
         global argument list.
-        Return zero if the arguments are invalid.
+        Return -1 if the arguments are invalid.
 
         Without arguments use the current window.
         With {winnr} only use this window in the current tab page.

--
Yukihiro Nakadaira - yukihiro.nakadaira <at> gmail.com

--
--
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" group.
To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscribe <at> googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
vim | 19 Aug 14:09 2014

Issue 250 in vim: Random US-International keyboard layout (dead keys) support

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 250 by John.Wel... <at> gmail.com: Random US-International keyboard  
layout (dead keys) support
http://code.google.com/p/vim/issues/detail?id=250

What steps will reproduce the problem?
1. OS: Win7 64 (32bit GVIm build)
2. Use US-International keyboard layout
3. In insert mode try to insert ' " ` ~ (hit the corresponding key followed  
by <SPACE>)

What is the expected output? What do you see instead?

Vim will fail to insert those characters on a fairly random basis.

The issue is perfectly described there:  
http://stackoverflow.com/questions/3937237/gvim-us-international-not-combining-dead-keys-with-space
and there:
http://vim.1045645.n5.nabble.com/dead-key-bug-td4358973.html

What version of the product are you using? On what operating system?
7.4.402 on Win7x64

Please provide any additional information below.

The issue seems random but dependent on the compiler and optimisations  
used, it first appeared with vim7. For years I have been dodging this issue  
by using  <at> wuyongwei builds but his latest 7.4.402 unfortunately exhibits  
the problem.

-- 
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

-- 
--

-- 
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" group.
To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscribe <at> googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

mattn | 19 Aug 12:44 2014
Picon

CmdUndefined

Hi list.

I implemented CmdUndefined autocmd that is listed in todo list.

https://gist.github.com/mattn/a9591869fb5d4139a2aa

http://marc.info/?l=vim-dev&m=100887940823495
http://marc.info/?l=vim-dev&m=110788023426405

I modified FuncUndefined also because add v:rhs to handle function-name or command-line that is not undefined.
I named the event argument v:rhs because Charles Campbell was pointed about this in e-mail on above. I put
v:lhs to avoid to make different index of list of variables when it need to add v:lhs really.
BTW, I hope to use v:lhs to handle sort()

Thanks.
- Yasuhiro Matsumoto

-- 
--

-- 
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" group.
To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscribe <at> googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

vim | 18 Aug 23:00 2014

Issue 249 in vim: Include icon in Windows context menu

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 249 by brian.ma... <at> betteradmin.com: Include icon in Windows  
context menu
http://code.google.com/p/vim/issues/detail?id=249

What version of the product are you using? On what operating system?
- GVim 7.4 (Aug 10, 2013)
- Windows 7, 64-bit

Please include the vim icon in the Windows context menu when right-clicking  
on a file to edit.  This would make it much easier to find the "Edit with  
Vim" entry, especially on context menus that have numerous other additional  
menus, many of which also already have icons.

-- 
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

-- 
--

-- 
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" group.
To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscribe <at> googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Dominique Pellé | 18 Aug 21:32 2014
Picon

Fixed typos vim tutor (fr, pt, zh)

Hi

Attached patch fixes typos in Bram's name in
French, Portuguese and Chinese tutor files.

Regards
Dominique

-- 
--

-- 
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" group.
To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscribe <at> googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
David Barnett | 18 Aug 19:35 2014
Picon

[Patch] Add go filetype with syntax

I've attached a patch to add a "go" filetype to vim, with syntax highlighting and filetype configuration. I
also updated the vim-ft-vroom repo URL to point to the renamed repo while I'm at it ("vim-ft.vroom"->"vim-ft-vroom").

These files originally came from the golang repository and are currently maintained at
https://github.com/google/vim-ft-go. The files are under a BSD license there, but I've obtained
approval from Google to include them in vim under the Vim license.

David

-- 
--

-- 
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" group.
To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscribe <at> googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
Attachment (vim-ft-go.patch): text/x-diff, 14 KiB
Marcin Szamotulski | 18 Aug 15:31 2014
Picon

add +cmd argument to :buffer command

Hello,

I was wondering for some time why the :buffer command does not allow for
+cmd argument (:help +cmd), like :edit does.  Though this is not
strictly necessary since one can always do
:buffer1|cmd
but the same is true for :edit command; one can do both:
:edit +cmd file
:edit file|cmd

I attach a patch as a proof of concept and to let it try.

Best regards,
Marcin Szamotulski
Attachment (buffer_editcmd.patch): text/x-diff, 919 bytes
Bram Moolenaar | 17 Aug 17:24 2014
Picon
Picon

Patch 7.4.410


Patch 7.4.410
Problem:    Fold does not open after search when there is a CmdwinLeave
	    autocommand.
Solution:   Restore KeyTyped. (Jacob Niehus)
Files:	    src/ex_getln.c

*** ../vim-7.4.409/src/ex_getln.c	2014-06-25 14:39:35.102348584 +0200
--- src/ex_getln.c	2014-08-17 17:21:38.811022992 +0200
***************
*** 6363,6368 ****
--- 6363,6371 ----
  #ifdef FEAT_RIGHTLEFT
      int			save_cmdmsg_rl = cmdmsg_rl;
  #endif
+ #ifdef FEAT_FOLDING
+     int			save_KeyTyped;
+ #endif

      /* Can't do this recursively.  Can't do it when typing a password. */
      if (cmdwin_type != 0
***************
*** 6497,6504 ****
--- 6500,6518 ----
      RedrawingDisabled = i;

  # ifdef FEAT_AUTOCMD
+ 
+ #  ifdef FEAT_FOLDING
+     save_KeyTyped = KeyTyped;
+ #  endif
+ 
      /* Trigger CmdwinLeave autocommands. */
      apply_autocmds(EVENT_CMDWINLEAVE, typestr, typestr, FALSE, curbuf);
+ 
+ #  ifdef FEAT_FOLDING
+     /* Restore KeyTyped in case it is modified by autocommands */
+     KeyTyped = save_KeyTyped;
+ #  endif
+ 
  # endif

      /* Restore the command line info. */
*** ../vim-7.4.409/src/version.c	2014-08-17 17:05:39.163057796 +0200
--- src/version.c	2014-08-17 17:19:33.011027554 +0200
***************
*** 743,744 ****
--- 743,746 ----
  {   /* Add new patch number below this line */
+ /**/
+     410,
  /**/

-- 
Have you heard about the new Beowulf cluster? It's so fast, it executes
an infinite loop in 6 seconds.

 /// Bram Moolenaar -- Bram <at> Moolenaar.net -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

-- 
--

-- 
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" group.
To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscribe <at> googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Bram Moolenaar | 17 Aug 17:06 2014
Picon
Picon

Patch 7.4.409


Patch 7.4.409
Problem:    Can't build with Perl on Fedora 20.
Solution:   Find xsubpp in another directory. (Michael Henry)
Files:	    src/Makefile, src/config.mk.in, src/configure.in,
	    src/auto/configure

*** ../vim-7.4.408/src/Makefile	2014-08-10 13:34:59.052785459 +0200
--- src/Makefile	2014-08-17 16:59:03.527072144 +0200
***************
*** 2477,2483 ****

  auto/if_perl.c: if_perl.xs
  	$(PERL) -e 'unless ( $$] >= 5.005 ) { for (qw(na defgv errgv)) { print "#define PL_$$_ $$_\n" }}' > $ <at> 
! 	$(PERL) $(PERLLIB)/ExtUtils/xsubpp -prototypes -typemap \
  	    $(PERLLIB)/ExtUtils/typemap if_perl.xs >> $ <at> 

  auto/osdef.h: auto/config.h osdef.sh osdef1.h.in osdef2.h.in
--- 2478,2484 ----

  auto/if_perl.c: if_perl.xs
  	$(PERL) -e 'unless ( $$] >= 5.005 ) { for (qw(na defgv errgv)) { print "#define PL_$$_ $$_\n" }}' > $ <at> 
! 	$(PERL) $(PERL_XSUBPP) -prototypes -typemap \
  	    $(PERLLIB)/ExtUtils/typemap if_perl.xs >> $ <at> 

  auto/osdef.h: auto/config.h osdef.sh osdef1.h.in osdef2.h.in
*** ../vim-7.4.408/src/config.mk.in	2014-06-12 18:03:24.936319000 +0200
--- src/config.mk.in	2014-08-17 16:56:28.371077771 +0200
***************
*** 48,58 ****
  MZSCHEME_OBJ	=  <at> MZSCHEME_OBJ <at> 
  MZSCHEME_CFLAGS	=  <at> MZSCHEME_CFLAGS <at> 
  MZSCHEME_PRO	=  <at> MZSCHEME_PRO <at> 
! MZSCHEME_EXTRA  =  <at> MZSCHEME_EXTRA <at> 
  MZSCHEME_MZC	=  <at> MZSCHEME_MZC <at> 

  PERL		=  <at> vi_cv_path_perl <at> 
  PERLLIB		=  <at> vi_cv_perllib <at> 
  PERL_LIBS	=  <at> PERL_LIBS <at> 
  SHRPENV		=  <at> shrpenv <at> 
  PERL_SRC	=  <at> PERL_SRC <at> 
--- 48,59 ----
  MZSCHEME_OBJ	=  <at> MZSCHEME_OBJ <at> 
  MZSCHEME_CFLAGS	=  <at> MZSCHEME_CFLAGS <at> 
  MZSCHEME_PRO	=  <at> MZSCHEME_PRO <at> 
! MZSCHEME_EXTRA	=  <at> MZSCHEME_EXTRA <at> 
  MZSCHEME_MZC	=  <at> MZSCHEME_MZC <at> 

  PERL		=  <at> vi_cv_path_perl <at> 
  PERLLIB		=  <at> vi_cv_perllib <at> 
+ PERL_XSUBPP	=  <at> vi_cv_perl_xsubpp <at> 
  PERL_LIBS	=  <at> PERL_LIBS <at> 
  SHRPENV		=  <at> shrpenv <at> 
  PERL_SRC	=  <at> PERL_SRC <at> 
*** ../vim-7.4.408/src/configure.in	2014-07-30 13:22:48.263621613 +0200
--- src/configure.in	2014-08-17 16:57:11.183076219 +0200
***************
*** 933,938 ****
--- 933,946 ----
        fi
        vi_cv_perllib=`$vi_cv_path_perl -MConfig -e 'print $Config{privlibexp}'`
        AC_SUBST(vi_cv_perllib)
+       vi_cv_perl_extutils=unknown_perl_extutils_path
+       for extutils_rel_path in ExtUtils vendor_perl/ExtUtils; do
+ 	xsubpp_path="$vi_cv_perllib/$extutils_rel_path/xsubpp"
+ 	if test -f "$xsubpp_path"; then
+ 	  vi_cv_perl_xsubpp="$xsubpp_path"
+ 	fi
+       done
+       AC_SUBST(vi_cv_perl_xsubpp)
        dnl Remove "-fno-something", it breaks using cproto.
        perlcppflags=`$vi_cv_path_perl -Mlib=$srcdir -MExtUtils::Embed \
  	      -e 'ccflags;perl_inc;print"\n"' | sed -e 's/-fno[[^ ]]*//'`
*** ../vim-7.4.408/src/auto/configure	2014-07-30 13:22:48.267621613 +0200
--- src/auto/configure	2014-08-17 16:58:43.047072887 +0200
***************
*** 687,692 ****
--- 687,693 ----
  PERL_OBJ
  PERL_SRC
  shrpenv
+ vi_cv_perl_xsubpp
  vi_cv_perllib
  vi_cv_path_perl
  MZSCHEME_MZC
***************
*** 5604,5609 ****
--- 5605,5618 ----
        fi
        vi_cv_perllib=`$vi_cv_path_perl -MConfig -e 'print $Config{privlibexp}'`

+       vi_cv_perl_extutils=unknown_perl_extutils_path
+       for extutils_rel_path in ExtUtils vendor_perl/ExtUtils; do
+ 	xsubpp_path="$vi_cv_perllib/$extutils_rel_path/xsubpp"
+ 	if test -f "$xsubpp_path"; then
+ 	  vi_cv_perl_xsubpp="$xsubpp_path"
+ 	fi
+       done
+ 
              perlcppflags=`$vi_cv_path_perl -Mlib=$srcdir -MExtUtils::Embed \
  	      -e 'ccflags;perl_inc;print"\n"' | sed -e 's/-fno[^ ]*//'`
              perllibs=`cd $srcdir; $vi_cv_path_perl -MExtUtils::Embed -e 'ldopts' | \
*** ../vim-7.4.408/src/version.c	2014-08-16 18:36:38.597993280 +0200
--- src/version.c	2014-08-17 16:53:34.135084090 +0200
***************
*** 743,744 ****
--- 743,746 ----
  {   /* Add new patch number below this line */
+ /**/
+     409,
  /**/

-- 
To define recursion, we must first define recursion.

 /// Bram Moolenaar -- Bram <at> Moolenaar.net -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

-- 
--

-- 
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" group.
To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscribe <at> googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

ZyX | 17 Aug 15:34 2014
Picon

[BUG] Lockvar does not work well with slices

1. List item locked with lockvar may be set using slice assignment:

        let l = [1, 2, 3, 4]
        lockvar! l
        unlockvar l[1]
        let l[0:1] = [0, 1]  " ^ Reports E741
        echo l  " [1, 2, 3, 4]
        let l[1:2] = [0, 1]
        echo l  " [1, 0, 1, 4]

2. Same for :unlet: if the first item in the slice was not locked :unlet may delete the whole slice even though
other values in the slice are locked.

-- 
--

-- 
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" group.
To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscribe <at> googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Gmane