Dominique Pelle wrote:
> Valgrind memory checker finds a bug in vim (when running
> test case vim7/src/testdir/test49.vim).
>
> Test 49 spawns several vim processes. Bug happens in
> some of the vim processes spawned by the main test49.vim
> script.
>
> Here is the bug reported by valgrind:
[...]
I can reproduce it with: :let v:statusmsg
Strangely the comment in the code suggests that one was aware of the
problem, but apparently a later change broke it again.
This patch should fix it. It's also better that listing variables
doesn't mess up v:statusmsg.
*** ../vim-7.1.064/src/eval.c Mon Aug 6 22:27:12 2007
--- src/eval.c Sun Aug 12 14:50:46 2007
***************
*** 18017,18023 ****
int type;
char_u *string;
{
! msg_attr(prefix, 0); /* don't use msg(), it overwrites "v:statusmsg" */
if (name != NULL) /* "a:" vars don't have a name stored */
msg_puts(name);
msg_putchar(' ');
--- 18017,18025 ----
int type;
char_u *string;
{
! /* don't use msg() or msg_attr() to avoid overwriting "v:statusmsg" */
! msg_start();
! msg_puts(prefix);
if (name != NULL) /* "a:" vars don't have a name stored */
msg_puts(name);
msg_putchar(' ');
--
"Microsoft is like Coke. It's a secret formula, all the money is from
distribution, and their goal is to get Coke everywhere. Open source is like
selling water. There are water companies like Perrier and Poland Spring, but
you're competing with something that's free." -- Carl Howe
/// Bram Moolenaar -- [EMAIL PROTECTED] -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ download, build and distribute -- http://www.A-A-P.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---
