Patch 7.4.266
Patch 7.4.266 Problem:Test 62 fails. Solution: Set the language to C. (Christian Brabandt) Files: src/testdir/test62.in *** ../vim-7.4.265/src/testdir/test62.in2013-07-14 13:37:12.0 +0200 --- src/testdir/test62.in 2014-04-29 11:50:50.676058452 +0200 *** *** 2,7 --- 2,8 STARTTEST :so small.vim + :lang C : Simple test for opening and closing a tab page :tabnew :let nr = tabpagenr() *** ../vim-7.4.265/src/version.c2014-04-24 17:12:29.584911764 +0200 --- src/version.c 2014-04-29 11:51:32.356057722 +0200 *** *** 736,737 --- 736,739 { /* Add new patch number below this line */ + /**/ + 266, /**/ -- I AM THANKFUL... ...for a lawn that needs mowing, windows that need cleaning and gutters that need fixing because it means I have a home. /// Bram Moolenaar -- b...@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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: [patch] test 62 fails because of wrong locale
Christian Brabandt wrote: I noticed an unexpected break of test62 (in the catch E474 part). This patch fixes it for me: diff --git a/src/testdir/test62.in b/src/testdir/test62.in --- a/src/testdir/test62.in +++ b/src/testdir/test62.in @@ -2,6 +2,7 @@ Tests for tab pages STARTTEST :so small.vim +:lang C : Simple test for opening and closing a tab page :tabnew :let nr = tabpagenr() For some reason, E474 isn't mentioned in my error message: Fehler beim Ausführen von : Ungültiges Argument OK. Maybe there is another solution, but if this works then let's use it. -- Bare feet magnetize sharp metal objects so they point upward from the floor -- especially in the dark. /// Bram Moolenaar -- b...@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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: Regression: Change mark '[ at end, not start of first line after gq.
Christian Brabandt wrote: On Do, 24 Apr 2014, Ingo Karkat wrote: Hello Vim developers, my plugins' automated test suite found another regression. When reformatting lines with gq{motion}, the start of change mark '[ does not point to the start of the first line [0, lnum, 1, 0] any more, but instead to after the end of the original first line [0, lnum, len(getline(lnum)) + 1, 0]. As gq processes entire lines, the start of the change should indeed be at column 1, as it used to. This scriptlet shows the discrepancy: :call setline(1, [\t\tO sodales, ludite, vos qui, attamen consulite per voster honur. Tua pulchra facies me fay planszer milies]) :1normal! gqj :echo getpos('[) Should yield [0, 1, 1, 0], but gives [0, 1, 29, 0]. Using the attached scriptlet, I've bisected this to the following patch: ,[ bad change ] | 7.4.178 the J command does not update '[ and '] marks ` I still see this in the latest 7.4.264 (HUGE build) on Linux/x64. -- regards, ingo Please check the following patch. I'll added a new test for this. Thanks. -- A fool learns from his mistakes, a wise man from someone else's. /// Bram Moolenaar -- b...@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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Patch 7.4.2
Patch 7.4.267 (after 7.4.178) Problem:The '[ mark is in the wrong position after gq. (Ingo Karkat) Solution: Add the setmark argument to do_join(). (Christian Brabandt) Files: src/testdir/Make_amiga.mak, src/testdir/Make_dos.mak, src/testdir/Make_ming.mak, src/testdir/Make_os2.mak, src/testdir/Make_vms.mms, src/testdir/Makefile, src/testdir/test_autoformat_join.in, src/testdir/test_autoformat_join.ok, src/Makefile, src/edit.c, src/ex_cmds.c, src/ex_docmd.c, src/normal.c, src/ops.c, src/proto/ops.pro *** ../vim-7.4.266/src/testdir/Make_amiga.mak 2014-04-02 19:54:58.275599459 +0200 --- src/testdir/Make_amiga.mak 2014-04-29 12:02:34.144046123 +0200 *** *** 36,41 --- 36,42 test94.out test95.out test96.out test97.out test98.out \ test99.out test100.out test101.out test102.out test103.out \ test104.out test105.out test106.out \ + test_autoformat_join.out \ test_eval.out \ test_options.out *** *** 160,164 --- 161,166 test104.out: test104.in test105.out: test105.in test106.out: test106.in + test_autoformat_join.out: test_autoformat_join.in test_eval.out: test_eval.in test_options.out: test_options.in *** ../vim-7.4.266/src/testdir/Make_dos.mak 2014-04-02 19:54:58.275599459 +0200 --- src/testdir/Make_dos.mak2014-04-29 12:02:45.128045931 +0200 *** *** 35,40 --- 35,41 test94.out test95.out test96.out test98.out test99.out \ test100.out test101.out test102.out test103.out test104.out \ test105.out test106.out \ + test_autoformat_join.out \ test_eval.out \ test_options.out *** ../vim-7.4.266/src/testdir/Make_ming.mak2014-04-02 19:54:58.275599459 +0200 --- src/testdir/Make_ming.mak 2014-04-29 12:02:54.020045775 +0200 *** *** 55,60 --- 55,61 test94.out test95.out test96.out test98.out test99.out \ test100.out test101.out test102.out test103.out test104.out \ test105.out test106.out \ + test_autoformat_join.out \ test_eval.out \ test_options.out *** ../vim-7.4.266/src/testdir/Make_os2.mak 2014-04-02 19:54:58.275599459 +0200 --- src/testdir/Make_os2.mak2014-04-29 12:03:03.384045611 +0200 *** *** 37,42 --- 37,43 test94.out test95.out test96.out test98.out test99.out \ test100.out test101.out test102.out test103.out test104.out \ test105.out test106.out \ + test_autoformat_join.out \ test_eval.out \ test_options.out *** ../vim-7.4.266/src/testdir/Make_vms.mms 2014-04-02 19:54:58.275599459 +0200 --- src/testdir/Make_vms.mms2014-04-29 12:03:12.200045456 +0200 *** *** 96,101 --- 96,102 test95.out test96.out test98.out test99.out \ test100.out test101.out test103.out test104.out \ test105.out test106.out \ +test_autoformat_join.out \ test_eval.out \ test_options.out *** ../vim-7.4.266/src/testdir/Makefile 2014-04-01 14:08:14.685074130 +0200 --- src/testdir/Makefile2014-04-29 12:02:09.152046561 +0200 *** *** 33,38 --- 33,40 test94.out test95.out test96.out test97.out test98.out \ test99.out test100.out test101.out test102.out test103.out \ test104.out test105.out test106.out \ + test_autoformat_join.out \ + test_eval.out \ test_options.out SCRIPTS_GUI = test16.out *** ../vim-7.4.266/src/testdir/test_autoformat_join.in 2014-04-29 12:12:57.324035202 +0200 --- src/testdir/test_autoformat_join.in 2014-04-29 12:00:35.360048205 +0200 *** *** 0 --- 1,23 + Tests for setting the '[,'] marks when joining lines. + + STARTTEST + :so small.vim + :/^\t\t/ + 0gqj + :let a=string(getpos('[)).'/'.string(getpos('])) + :/^This line/;'}-join + :let b=string(getpos('[)).'/'.string(getpos('])) + :$put ='First test: Start/End '.string(a) + :$put ='Second test: Start/End '.string(b) + :/^\t\t/,$wq! test.out + ENDTEST + + + O sodales, ludite, vos qui + attamen consulite per voster honur. Tua pulchra facies me fay planszer milies + + This line. + Should be joined with the next line + and with this line + + Results: *** ../vim-7.4.266/src/testdir/test_autoformat_join.ok 2014-04-29 12:12:57.332035201 +0200 --- src/testdir/test_autoformat_join.ok 2014-04-29 12:00:35.360048205 +0200 *** *** 0 --- 1,8 + O sodales, ludite, vos qui attamen consulite per voster honur. + Tua pulchra facies me fay planszer milies + + This line. Should be joined with the next
Re: Regression: Change mark '[ at end, not start of first line after gq.
On 28-Apr-2014 23:05 +0200, Christian Brabandt wrote: On Do, 24 Apr 2014, Ingo Karkat wrote: Hello Vim developers, my plugins' automated test suite found another regression. When reformatting lines with gq{motion}, the start of change mark '[ does not point to the start of the first line [0, lnum, 1, 0] any more, but instead to after the end of the original first line [0, lnum, len(getline(lnum)) + 1, 0]. As gq processes entire lines, the start of the change should indeed be at column 1, as it used to. This scriptlet shows the discrepancy: :call setline(1, [\t\tO sodales, ludite, vos qui, attamen consulite per voster honur. Tua pulchra facies me fay planszer milies]) :1normal! gqj :echo getpos('[) Should yield [0, 1, 1, 0], but gives [0, 1, 29, 0]. Using the attached scriptlet, I've bisected this to the following patch: ,[ bad change ] | 7.4.178 the J command does not update '[ and '] marks ` I still see this in the latest 7.4.264 (HUGE build) on Linux/x64. -- regards, ingo Please check the following patch. I'll added a new test for this. Good job (as always)! I can confirm that this (patch 7.4.267) fixes the problem for me and didn't create any new issues in my plugins' test suite. -- thanks, ingo -- -- 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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Patch 7.4.268
Patch 7.4.268 Problem:Using exists() on a funcref for a script-local function does not work. Solution: Translate SNR to the special byte sequence. Add a test. Files: src/eval.c, src/testdir/test_eval.in, src/testdir/test_eval.ok, src/testdir/test_eval_func.vim, Filelist *** ../vim-7.4.267/src/eval.c 2014-04-24 17:12:29.580911764 +0200 --- src/eval.c 2014-04-29 13:49:26.759933737 +0200 *** *** 22275,22280 --- 22275,22288 { name = vim_strsave(name); *pp = end; + if (STRNCMP(name, SNR, 5) == 0) + { + /* Change SNR to the byte sequence. */ + name[0] = K_SPECIAL; + name[1] = KS_EXTRA; + name[2] = (int)KE_SNR; + mch_memmove(name + 3, name + 5, STRLEN(name + 5) + 1); + } goto theend; } *** ../vim-7.4.267/src/testdir/test_eval.in 2014-04-24 17:12:29.584911764 +0200 --- src/testdir/test_eval.in2014-04-29 13:59:09.495923525 +0200 *** *** 180,185 --- 180,188 :echo g:Foo(2) :echo Foo(3) : + : script-local function used in Funcref must exist. + :so test_eval_func.vim + : :/^start:/+1,$wq! test.out : vim: et ts=4 isk-=\: fmr=???,??? :call getchar() *** ../vim-7.4.267/src/testdir/test_eval.ok 2014-04-24 17:12:29.584911764 +0200 --- src/testdir/test_eval.ok2014-04-29 13:59:39.551922998 +0200 *** *** 341,343 --- 341,347 called Foo(1) called Foo(2) called Foo(3) + s:Testje exists: 0 + func s:Testje exists: 1 + Bar exists: 1 + func Bar exists: 1 *** ../vim-7.4.267/src/testdir/test_eval_func.vim 2014-04-29 14:02:08.415920389 +0200 --- src/testdir/test_eval_func.vim 2014-04-29 13:57:34.503925189 +0200 *** *** 0 --- 1,10 + Vim script used in test_eval.in. Needed for script-local function. + + func! s:Testje() + return foo + endfunc + let Bar = function('s:Testje') + $put ='s:Testje exists: ' . exists('s:Testje') + $put ='func s:Testje exists: ' . exists('*s:Testje') + $put ='Bar exists: ' . exists('Bar') + $put ='func Bar exists: ' . exists('*Bar') *** ../vim-7.4.267/Filelist 2014-04-05 21:59:35.939178415 +0200 --- Filelist2014-04-29 13:58:40.207924038 +0200 *** *** 88,93 --- 88,94 src/testdir/test60.vim \ src/testdir/test83-tags? \ src/testdir/test77a.com \ + src/testdir/test_*.vim \ src/testdir/python2/*.py \ src/testdir/python3/*.py \ src/testdir/pythonx/*.py \ *** ../vim-7.4.267/src/version.c2014-04-29 12:15:22.860032651 +0200 --- src/version.c 2014-04-29 13:47:32.935935732 +0200 *** *** 736,737 --- 736,739 { /* Add new patch number below this line */ + /**/ + 268, /**/ -- I AM THANKFUL... ...for the piles of laundry and ironing because it means I have plenty of clothes to wear. /// Bram Moolenaar -- b...@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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Patch 7.4.269
Patch 7.4.269 Problem:CTRL-U in Insert mode does not work after using a cursor key. (Pine Wu) Solution: Use the original insert start position. (Christian Brabandt) Files: src/edit.c, src/testdir/test29.in, src/testdir/test29.ok *** ../vim-7.4.268/src/edit.c 2014-04-29 12:15:22.852032651 +0200 --- src/edit.c 2014-04-29 14:44:07.867876234 +0200 *** *** 8760,8767 ((curwin-w_cursor.lnum == 1 curwin-w_cursor.col == 0) || (!can_bs(BS_START) (arrow_used ! || (curwin-w_cursor.lnum == Insstart.lnum !curwin-w_cursor.col = Insstart.col))) || (!can_bs(BS_INDENT) !arrow_used ai_col 0 curwin-w_cursor.col = ai_col) || (!can_bs(BS_EOL) curwin-w_cursor.col == 0 --- 8760,8767 ((curwin-w_cursor.lnum == 1 curwin-w_cursor.col == 0) || (!can_bs(BS_START) (arrow_used ! || (curwin-w_cursor.lnum == Insstart_orig.lnum !curwin-w_cursor.col = Insstart_orig.col))) || (!can_bs(BS_INDENT) !arrow_used ai_col 0 curwin-w_cursor.col = ai_col) || (!can_bs(BS_EOL) curwin-w_cursor.col == 0 *** *** 8812,8819 */ if (curwin-w_cursor.col == 0) { ! lnum = Insstart.lnum; ! if (curwin-w_cursor.lnum == Insstart.lnum #ifdef FEAT_RIGHTLEFT || revins_on #endif --- 8812,8819 */ if (curwin-w_cursor.col == 0) { ! lnum = Insstart_orig.lnum; ! if (curwin-w_cursor.lnum == lnum #ifdef FEAT_RIGHTLEFT || revins_on #endif *** *** 8822,8829 if (u_save((linenr_T)(curwin-w_cursor.lnum - 2), (linenr_T)(curwin-w_cursor.lnum + 1)) == FAIL) return FALSE; ! --Insstart.lnum; ! Insstart.col = MAXCOL; } /* * In replace mode: --- 8822,8829 if (u_save((linenr_T)(curwin-w_cursor.lnum - 2), (linenr_T)(curwin-w_cursor.lnum + 1)) == FAIL) return FALSE; ! --Insstart_orig.lnum; ! Insstart_orig.col = MAXCOL; } /* * In replace mode: *** *** 8981,8989 while (vcol want_vcol) { /* Remember the first char we inserted */ ! if (curwin-w_cursor.lnum == Insstart.lnum ! curwin-w_cursor.col Insstart.col) ! Insstart.col = curwin-w_cursor.col; #ifdef FEAT_VREPLACE if (State VREPLACE_FLAG) --- 8981,8989 while (vcol want_vcol) { /* Remember the first char we inserted */ ! if (curwin-w_cursor.lnum == Insstart_orig.lnum ! curwin-w_cursor.col Insstart_orig.col) ! Insstart_orig.col = curwin-w_cursor.col; #ifdef FEAT_VREPLACE if (State VREPLACE_FLAG) *** *** 9071,9078 revins_on || #endif (curwin-w_cursor.col mincol ! (curwin-w_cursor.lnum != Insstart.lnum !|| curwin-w_cursor.col != Insstart.col))); did_backspace = TRUE; } #ifdef FEAT_SMARTINDENT --- 9071,9078 revins_on || #endif (curwin-w_cursor.col mincol ! (curwin-w_cursor.lnum != Insstart_orig.lnum !|| curwin-w_cursor.col != Insstart_orig.col))); did_backspace = TRUE; } #ifdef FEAT_SMARTINDENT *** *** 9090,9098 AppendCharToRedobuff(c); /* If deleted before the insertion point, adjust it */ ! if (curwin-w_cursor.lnum == Insstart.lnum ! curwin-w_cursor.col Insstart.col) ! Insstart.col = curwin-w_cursor.col; /* vi behaviour: the cursor moves backward but the character that * was there remains visible --- 9090,9098 AppendCharToRedobuff(c); /* If deleted before the insertion point, adjust it */ ! if (curwin-w_cursor.lnum == Insstart_orig.lnum ! curwin-w_cursor.col Insstart_orig.col) ! Insstart_orig.col = curwin-w_cursor.col; /* vi behaviour: the cursor moves backward but the character that * was there remains visible *** ../vim-7.4.268/src/testdir/test29.in2012-06-13 13:48:26.0 +0200 --- src/testdir/test29.in 2014-04-29 14:31:23.619889628 +0200 *** *** 102,107 --- 102,135 } STARTTEST + :
Patch 7.4.270
Patch 7.4.270 Problem:Comparing pointers instead of the string they point to. Solution: Use strcmp(). (Ken Takata) Files: src/gui_gtk_x11.c *** ../vim-7.4.269/src/gui_gtk_x11.c2014-04-06 21:08:41.311360470 +0200 --- src/gui_gtk_x11.c 2014-04-29 15:02:35.931856814 +0200 *** *** 3142,3151 gnome_program_init(VIMPACKAGE, VIM_VERSION_SHORT, LIBGNOMEUI_MODULE, gui_argc, gui_argv, NULL); # if defined(FEAT_FLOAT) defined(LC_NUMERIC) ! /* Make sure strtod() uses a decimal point, not a comma. Gnome init !* may change it. */ ! if (setlocale(LC_NUMERIC, NULL) != (char *) C) ! setlocale(LC_NUMERIC, C); # endif } #endif --- 3142,3155 gnome_program_init(VIMPACKAGE, VIM_VERSION_SHORT, LIBGNOMEUI_MODULE, gui_argc, gui_argv, NULL); # if defined(FEAT_FLOAT) defined(LC_NUMERIC) ! { ! char *p = setlocale(LC_NUMERIC, NULL); ! ! /* Make sure strtod() uses a decimal point, not a comma. Gnome !* init may change it. */ ! if (p == NULL || strcmp(p, C) != 0) ! setlocale(LC_NUMERIC, C); ! } # endif } #endif *** ../vim-7.4.269/src/version.c2014-04-29 14:44:31.519875819 +0200 --- src/version.c 2014-04-29 14:59:21.799860216 +0200 *** *** 736,737 --- 736,739 { /* Add new patch number below this line */ + /**/ + 270, /**/ -- From know your smileys: :-| :-| Deja' vu! /// Bram Moolenaar -- b...@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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: [patch] The misreading of tabediting Session.vim
Nobuhiro Takasaki wrote: I use a lot of mksession. I fixed an issue that occurs. Reproduce the problem: edit Makefile exe 10 normal! zt 20 tabnew tabnext 1 Open the Makefile, jump to line number 10, go to the top of the screen that line, line number 20 to move the cursor, open the tab, go back to the original screen. Scroll position will be shifted tabnew exists. The result is correct if there is no tabnew. I enclose a short patch. I cannot reproduce this problem. You mention mksession, but the steps to reproduce the problem do not include a session. Please start with vim -u NONE so that none of your settings affect what happens. -- From know your smileys: :-| :-| Deja' vu! /// Bram Moolenaar -- b...@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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Patch 7.4.271
Patch 7.4.271 Problem:Compiler warning on 64 bit windows. Solution: Add type cast. (Mike Williams) Files: src/ops.c *** ../vim-7.4.270/src/ops.c2014-04-29 12:15:22.860032651 +0200 --- src/ops.c 2014-04-29 15:54:23.055802359 +0200 *** *** 6559,6565 { for (ss = (char_u **) str; *ss != NULL; ++ss, ++lnum) { ! i = STRLEN(*ss); pp[lnum] = vim_strnsave(*ss, i); if (i maxlen) maxlen = i; --- 6559,6565 { for (ss = (char_u **) str; *ss != NULL; ++ss, ++lnum) { ! i = (long)STRLEN(*ss); pp[lnum] = vim_strnsave(*ss, i); if (i maxlen) maxlen = i; *** ../vim-7.4.270/src/version.c2014-04-29 15:11:39.783847283 +0200 --- src/version.c 2014-04-29 15:52:54.083803919 +0200 *** *** 736,737 --- 736,739 { /* Add new patch number below this line */ + /**/ + 271, /**/ -- From know your smileys: :.-( Crying /// Bram Moolenaar -- b...@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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Patch 7.4.272
Patch 7.4.272 Problem:Using just $ does not cause an error message. Solution: Check for empty environment variable name. (Christian Brabandt) Files: src/eval.c, src/testdir/test_eval.in, src/testdir/test_eval.ok *** ../vim-7.4.271/src/eval.c 2014-04-29 14:02:42.543919791 +0200 --- src/eval.c 2014-04-29 17:33:40.575697949 +0200 *** *** 7798,7804 * Get the value of an environment variable. * arg is pointing to the '$'. It is advanced to after the name. * If the environment variable was not set, silently assume it is empty. ! * Always return OK. */ static int get_env_tv(arg, rettv, evaluate) --- 7798,7804 * Get the value of an environment variable. * arg is pointing to the '$'. It is advanced to after the name. * If the environment variable was not set, silently assume it is empty. ! * Return FAIL if the name is invalid. */ static int get_env_tv(arg, rettv, evaluate) *** *** 7817,7848 len = get_env_len(arg); if (evaluate) { ! if (len != 0) { ! cc = name[len]; ! name[len] = NUL; ! /* first try vim_getenv(), fast for normal environment vars */ ! string = vim_getenv(name, mustfree); ! if (string != NULL *string != NUL) ! { ! if (!mustfree) ! string = vim_strsave(string); ! } ! else ! { ! if (mustfree) ! vim_free(string); ! /* next try expanding things like $VIM and ${HOME} */ ! string = expand_env_save(name - 1); ! if (string != NULL *string == '$') ! { ! vim_free(string); ! string = NULL; ! } } - name[len] = cc; } rettv-v_type = VAR_STRING; rettv-vval.v_string = string; } --- 7817,7849 len = get_env_len(arg); if (evaluate) { ! if (len == 0) !return FAIL; /* can't be an environment variable */ ! ! cc = name[len]; ! name[len] = NUL; ! /* first try vim_getenv(), fast for normal environment vars */ ! string = vim_getenv(name, mustfree); ! if (string != NULL *string != NUL) { ! if (!mustfree) ! string = vim_strsave(string); ! } ! else ! { ! if (mustfree) ! vim_free(string); ! /* next try expanding things like $VIM and ${HOME} */ ! string = expand_env_save(name - 1); ! if (string != NULL *string == '$') ! { ! vim_free(string); ! string = NULL; } } + name[len] = cc; + rettv-v_type = VAR_STRING; rettv-vval.v_string = string; } *** ../vim-7.4.271/src/testdir/test_eval.in 2014-04-29 14:02:42.543919791 +0200 --- src/testdir/test_eval.in2014-04-29 17:35:27.243696080 +0200 *** *** 183,188 --- 183,195 : script-local function used in Funcref must exist. :so test_eval_func.vim : + : Using $ instead of '$' must give an error + :try + : call append($, 'foobar') + :catch + :$put =v:exception + :endtry + : :/^start:/+1,$wq! test.out : vim: et ts=4 isk-=\: fmr=???,??? :call getchar() *** ../vim-7.4.271/src/testdir/test_eval.ok 2014-04-29 14:02:42.543919791 +0200 --- src/testdir/test_eval.ok2014-04-29 17:36:41.451694779 +0200 *** *** 345,347 --- 345,348 func s:Testje exists: 1 Bar exists: 1 func Bar exists: 1 + Vim(call):E116: Invalid arguments for function append *** ../vim-7.4.271/src/version.c2014-04-29 15:55:39.443801021 +0200 --- src/version.c 2014-04-29 17:31:54.203699814 +0200 *** *** 736,737 --- 736,739 { /* Add new patch number below this line */ + /**/ + 272, /**/ -- From know your smileys: C=};*{)) Drunk, devilish chef with a toupee in an updraft, a mustache, and a double chin /// Bram Moolenaar -- b...@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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: Issue 220 in vim: missing support for RR types TLSA and SSHFP in bindzone.vim
Updates: Status: WontFix Comment #4 on issue 220 by brammool...@gmail.com: missing support for RR types TLSA and SSHFP in bindzone.vim http://code.google.com/p/vim/issues/detail?id=220 Close on creators requet. -- 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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Patch 7.4.273
Patch 7.4.273 Problem:make autoconf and make reconfig may first run configure and then remove the output. Solution: Add these targets to the exceptions. (Ken Takata) Files: src/Makefile *** ../vim-7.4.272/src/Makefile 2014-04-29 12:15:22.852032651 +0200 --- src/Makefile2014-04-29 19:58:23.611545773 +0200 *** *** 1670,1676 ! grep -x -F 'ac_cv_env_CFLAGS_value=$(CFLAGS)' auto/config.cache /dev/null; then \ rm auto/config.cache; \ fi ! if test X$(MAKECMDGOALS) != Xclean -a X$(MAKECMDGOALS) != Xdistclean; then \ GUI_INC_LOC=$(GUI_INC_LOC) GUI_LIB_LOC=$(GUI_LIB_LOC) \ CC=$(CC) CPPFLAGS=$(CPPFLAGS) CFLAGS=$(CFLAGS) \ LDFLAGS=$(LDFLAGS) $(CONF_SHELL) srcdir=$(srcdir) \ --- 1670,1679 ! grep -x -F 'ac_cv_env_CFLAGS_value=$(CFLAGS)' auto/config.cache /dev/null; then \ rm auto/config.cache; \ fi ! if test X$(MAKECMDGOALS) != Xclean \ ! -a X$(MAKECMDGOALS) != Xdistclean \ ! -a X$(MAKECMDGOALS) != Xautoconf \ ! -a X$(MAKECMDGOALS) != Xreconfig; then \ GUI_INC_LOC=$(GUI_INC_LOC) GUI_LIB_LOC=$(GUI_LIB_LOC) \ CC=$(CC) CPPFLAGS=$(CPPFLAGS) CFLAGS=$(CFLAGS) \ LDFLAGS=$(LDFLAGS) $(CONF_SHELL) srcdir=$(srcdir) \ *** ../vim-7.4.272/src/version.c2014-04-29 17:41:18.351689927 +0200 --- src/version.c 2014-04-29 19:59:21.895544751 +0200 *** *** 736,737 --- 736,739 { /* Add new patch number below this line */ + /**/ + 273, /**/ -- From know your smileys: |-) Chinese |-( Chinese and doesn't like these kind of jokes /// Bram Moolenaar -- b...@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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: _dd doesn't reset v:register
Hi, Christian, Can you please come up with a mapping that shows the wrong behaviour? I made this example: https://gist.github.com/AndrewRadev/11408444 It's supposed to paste some text and comment it in the process. It's kind of silly, but a realistic enough example. If you `_dd` anything and then immediately perform a `gp`, nothing happens, since the `v:register` variable is set to `_`. If you've managed ot fix the issue, it should paste normally, from the default register. Best Regards, Andrew. -- -- 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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: _dd doesn't reset v:register
Hi Andrew! On Di, 29 Apr 2014, Andrew wrote: Hi, Christian, Can you please come up with a mapping that shows the wrong behaviour? I made this example: https://gist.github.com/AndrewRadev/11408444 It's supposed to paste some text and comment it in the process. It's kind of silly, but a realistic enough example. If you `_dd` anything and then immediately perform a `gp`, nothing happens, since the `v:register` variable is set to `_`. If you've managed ot fix the issue, it should paste normally, from the default register. Thanks. I think this one fixes it: diff --git a/src/normal.c b/src/normal.c --- a/src/normal.c +++ b/src/normal.c @@ -1263,6 +1263,10 @@ getcount: normal_end: msg_nowait = FALSE; +#ifdef FEAT_EVAL +/* reset v:register */ +set_reg_var(0); +#endif /* Reset finish_op, in case it was set */ #ifdef CURSOR_SHAPE (I hope, it doesn't have any side effect). Best, Christian -- -- 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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: _dd doesn't reset v:register
Hi, Christian, Thanks. I think this one fixes it. Yes, it seems like it does. I just applied it to my local vim and everything seems to work fine. (I hope, it doesn't have any side effect). So do I :). Thanks for the fix! I guess all that's left is to wait until it's merged. Best Regards, Andrew. -- -- 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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: _dd doesn't reset v:register
Christian Brabandt wrote: Hi Andrew! On Di, 29 Apr 2014, Andrew wrote: Hi, Christian, Can you please come up with a mapping that shows the wrong behaviour? I made this example: https://gist.github.com/AndrewRadev/11408444 It's supposed to paste some text and comment it in the process. It's kind of silly, but a realistic enough example. If you `_dd` anything and then immediately perform a `gp`, nothing happens, since the `v:register` variable is set to `_`. If you've managed ot fix the issue, it should paste normally, from the default register. Thanks. I think this one fixes it: diff --git a/src/normal.c b/src/normal.c --- a/src/normal.c +++ b/src/normal.c @@ -1263,6 +1263,10 @@ getcount: normal_end: msg_nowait = FALSE; +#ifdef FEAT_EVAL +/* reset v:register */ +set_reg_var(0); +#endif /* Reset finish_op, in case it was set */ #ifdef CURSOR_SHAPE (I hope, it doesn't have any side effect). Well, if one first does add and then _dd, it's a bit unexpected that v:register is then '' and not '_' or 'a'. It's more complicated to remember the previous value, then set v:register to '_' while the operation is being executed, then restore the previous value. An alternative would be to have another variable that holds the last register that isn't the black hole register. Not sure how useful that would be. -- From know your smileys: :-*A big kiss! /// Bram Moolenaar -- b...@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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: Issue 117 in vim: boolean to keep the sign column open at all times
Comment #5 on issue 117 by sheme...@gmail.com: boolean to keep the sign column open at all times http://code.google.com/p/vim/issues/detail?id=117 It is awesome idea. It's pretty annoying to have text shifting all the time -- 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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: [patch] The misreading of tabediting Session.vim
I have minimized the problem Session.vim out under me. I've prepared a text file with line numbers here. Please do this operations: $ vim lines.txt :tabnew :tabnext 1 50j :mksession! :qa $ vim -u NONE -S Line number 50 will be the top of the screen. Line number 01 at the top of the screen if there is no tabnew. It is difficult because I am using the tab copen. Thanks. Nobuhiro Takasaki -- -- 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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout. 01 02 03 04 05 06 07 08 09 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50