[patch] doc fixes in help.txt

2020-04-04 Fir de Conversatie Dominique Pellé
Hi

Please find attached doc fixes for help.txt.

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+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/CAON-T_jw2qGGN%3DaEpdM4nQwzHZ9wE%2BMwA-r4hmQ%2BzozaPUor-Q%40mail.gmail.com.
diff --git a/runtime/doc/help.txt b/runtime/doc/help.txt
index f2b78466e..1c96bff19 100644
--- a/runtime/doc/help.txt
+++ b/runtime/doc/help.txt
@@ -31,7 +31,7 @@ Get specific help:  It is possible to go directly to whatever you want help
 		help entries for "word".
 		Or use ":helpgrep word". |:helpgrep|
 
-  Getting started:  Do the Vim tutor, a 20 minute interactive training for the
+  Getting started:  Do the Vim tutor, a 20 minutes interactive training for the
 		basic commands, see |vimtutor|.
 		Read the user manual from start to end: |usr_01.txt|
 
@@ -144,7 +144,7 @@ Special issues ~
 |remote.txt|	using Vim as a server or client
 |term.txt|	using different terminals and mice
 |terminal.txt|	Terminal window support
-|popup.txt|	popop window support
+|popup.txt|	popup window support
 
 Programming language support ~
 |indent.txt|	automatic indenting for C and other languages


E1055 error code is used for two different error conditions

2020-04-04 Fir de Conversatie Yegappan Lakshmanan
Hi,

The E1055 error code is used in two different places:

userfunc.c:emsg(_("E1055: Missing name after ..."));
vim9compile.c: emsg(_("E1055: This Vim is not compiled with float
support"));

- 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

--- 
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.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/CAAW7x7%3DuSjRNF_9RREZsZHM2AVp83qQ2mcR9z6FNJi2frRraTA%40mail.gmail.com.


Patch 8.2.0510

2020-04-04 Fir de Conversatie Bram Moolenaar


Patch 8.2.0510
Problem:Coverity complains about using uninitialized variable.
Solution:   Assign a value to "scol".  Move code inside NULL check.
Files:  src/beval.c, src/popupwin.c


*** ../vim-8.2.0509/src/beval.c 2019-11-30 22:40:44.0 +0100
--- src/beval.c 2020-04-04 14:47:28.248431226 +0200
***
*** 110,115 
--- 110,117 
lbuf = vim_strnsave(lbuf, len);
}
}
+   else
+   scol = col;
  
if (winp != NULL)
*winp = wp;
*** ../vim-8.2.0509/src/popupwin.c  2020-04-02 18:50:42.423773112 +0200
--- src/popupwin.c  2020-04-04 14:49:39.855963305 +0200
***
*** 798,813 
if (*str != NUL)
wp->w_border_char[i] = mb_ptr2char(str);
}
!   }
!   if (list->lv_len == 1)
!   for (i = 1; i < 8; ++i)
!   wp->w_border_char[i] = wp->w_border_char[0];
!   if (list->lv_len == 2)
!   {
!   for (i = 4; i < 8; ++i)
!   wp->w_border_char[i] = wp->w_border_char[1];
!   for (i = 1; i < 4; ++i)
!   wp->w_border_char[i] = wp->w_border_char[0];
}
}
  }
--- 798,813 
if (*str != NUL)
wp->w_border_char[i] = mb_ptr2char(str);
}
!   if (list->lv_len == 1)
!   for (i = 1; i < 8; ++i)
!   wp->w_border_char[i] = wp->w_border_char[0];
!   if (list->lv_len == 2)
!   {
!   for (i = 4; i < 8; ++i)
!   wp->w_border_char[i] = wp->w_border_char[1];
!   for (i = 1; i < 4; ++i)
!   wp->w_border_char[i] = wp->w_border_char[0];
!   }
}
}
  }
*** ../vim-8.2.0509/src/version.c   2020-04-04 14:00:34.197098267 +0200
--- src/version.c   2020-04-04 14:50:00.063891507 +0200
***
*** 740,741 
--- 740,743 
  {   /* Add new patch number below this line */
+ /**/
+ 510,
  /**/

-- 
BRIDGEKEEPER: What is your favorite editor?
GAWAIN:   Emacs ...  No, Viiimmm!
   "Monty Python and the Holy editor wars" PYTHON (MONTY) SOFTWARE LTD

 /// 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.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/202004041251.034Cp8kn006169%40masaka.moolenaar.net.


Patch 8.2.0509

2020-04-04 Fir de Conversatie Bram Moolenaar


Patch 8.2.0509
Problem:various code is not properly tested.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5871)
Files:  src/main.c, src/testdir/check.vim, src/testdir/shared.vim,
src/testdir/term_util.vim, src/testdir/test_clientserver.vim,
src/testdir/test_ex_mode.vim, src/testdir/test_expand.vim,
src/testdir/test_functions.vim, src/testdir/test_options.vim,
src/testdir/test_startup.vim, src/testdir/test_textformat.vim,
src/testdir/test_trycatch.vim, src/testdir/test_viminfo.vim


*** ../vim-8.2.0508/src/main.c  2020-02-14 13:21:55.646197062 +0100
--- src/main.c  2020-04-04 13:36:02.729150119 +0200
***
*** 1966,1972 
  #endif
  }
  
! // Checking for "ex" here may catch some weir names, such as "vimex" or
  // "viewex", we assume the user knows that.
  if (STRNICMP(initstr, "ex", 2) == 0)
  {
--- 1966,1972 
  #endif
  }
  
! // Checking for "ex" here may catch some weird names, such as "vimex" or
  // "viewex", we assume the user knows that.
  if (STRNICMP(initstr, "ex", 2) == 0)
  {
*** ../vim-8.2.0508/src/testdir/check.vim   2020-03-26 22:16:44.307582067 
+0100
--- src/testdir/check.vim   2020-04-04 13:36:02.729150119 +0200
***
*** 133,135 
--- 133,145 
  throw 'Skipped: cannot run test as root'
endif
  endfunc
+ 
+ " Command to check that the current language is English
+ command CheckEnglish call CheckEnglish()
+ func CheckEnglish()
+   if v:lang != "C" && v:lang !~ '^[Ee]n'
+   throw 'Skipped: only works in English language environment'
+   endif
+ endfunc
+ 
+ " vim: shiftwidth=2 sts=2 expandtab
*** ../vim-8.2.0508/src/testdir/shared.vim  2020-01-30 18:24:46.997204019 
+0100
--- src/testdir/shared.vim  2020-04-04 13:36:02.729150119 +0200
***
*** 337,339 
--- 337,341 
endif
return v:false
  endfunc
+ 
+ " vim: shiftwidth=2 sts=2 expandtab
*** ../vim-8.2.0508/src/testdir/term_util.vim   2020-03-18 21:10:41.104567492 
+0100
--- src/testdir/term_util.vim   2020-04-04 13:36:02.729150119 +0200
***
*** 82,87 
--- 82,89 
  let cols = term_getsize(buf)[1]
endif
  
+   call term_wait(buf)
+ 
" Wait for "All" or "Top" of the ruler to be shown in the last line or in
" the status line of the last window. This can be quite slow (e.g. when
" using valgrind).
***
*** 113,115 
--- 115,119 
call WaitForAssert({-> assert_equal("finished", term_getstatus(a:buf))})
only!
  endfunc
+ 
+ " vim: shiftwidth=2 sts=2 expandtab
*** ../vim-8.2.0508/src/testdir/test_clientserver.vim   2020-03-30 
19:30:07.129542925 +0200
--- src/testdir/test_clientserver.vim   2020-04-04 13:36:02.729150119 +0200
***
*** 6,16 
  
  source shared.vim
  
! func Test_client_server()
!   let cmd = GetVimCommand()
!   if cmd == ''
! return
!   endif
if has('x11')
  if empty($DISPLAY)
throw 'Skipped: $DISPLAY is not set'
--- 6,12 
  
  source shared.vim
  
! func Check_X11_Connection()
if has('x11')
  if empty($DISPLAY)
throw 'Skipped: $DISPLAY is not set'
***
*** 19,29 
call remote_send('xxx', '')
  catch
if v:exception =~ 'E240:'
!   throw 'Skipped: no connection to the X server'
endif
" ignore other errors
  endtry
endif
  
let name = 'XVIMTEST'
let cmd .= ' --servername ' . name
--- 15,33 
call remote_send('xxx', '')
  catch
if v:exception =~ 'E240:'
! throw 'Skipped: no connection to the X server'
endif
" ignore other errors
  endtry
endif
+ endfunc
+ 
+ func Test_client_server()
+   let cmd = GetVimCommand()
+   if cmd == ''
+ return
+   endif
+   call Check_X11_Connection()
  
let name = 'XVIMTEST'
let cmd .= ' --servername ' . name
***
*** 72,77 
--- 76,85 
endif
let g:testvar = 'myself'
call assert_equal('myself', remote_expr(v:servername, 'testvar'))
+   call remote_send(v:servername, ":let g:testvar2 = 75\")
+   call feedkeys('', 'x')
+   call assert_equal(75, g:testvar2)
+   call assert_fails('let v = remote_expr(v:servername, "/2")', 'E449:')
  
call remote_send(name, ":call server2client(expand(''), 'got 
it')\", 'g:myserverid')
call assert_equal('got it', g:myserverid->remote_read(2))
***
*** 92,97 
--- 100,154 
call assert_equal('another', g:peek_result)
call assert_equal('another', remote_read(g:myserverid, 2))
  
+   if !has('gui_running')
+ " In GUI vim, the following tests display a dialog box
+ 
+ let cmd = GetVimProg() .. ' --servername ' .. name
+ 
+ " Run a separate instance to send a command to the server
+ call remote_expr(name, 'execute("only")')
+ call system(cmd .. ' --remote-send ":new Xfile"')
+ call assert_equal('2', remote_expr(name, 'winnr("$")'))
+