Hi Bram and Vim developers,

2015/4/22(Wed) 2:12:31 UTC+9 Bram Moolenaar:
> David Bürgin wrote:
> 
> > On Tuesday, April 21, 2015 at 6:35:26 PM UTC+2, Bram Moolenaar wrote:
> > > Patch 7.4.710
> > > Problem:    It is not possible to make spaces visibible in list mode.
> > > Solution:   Add the "space" item to 'listchars'. (David Bürgin, issue 350)
> > > Files:      runtime/doc/options.txt, src/globals.h, src/message.h,
> > >             src/screen.c, src/testdir/test_listchars.in,
> > >             src/testdir/test_listchars.ok, 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
> > 
> > Well, three cheers to that!
> > 
> > Unfortunately one hunk in option.c is missing so it doesn't work yet.
> 
> Oops, was missing from the list of files.

test_listchars FAILED.

7.4.710 patch only correspond to the ':set list'.
Does not correspond to the ':list'.

But test_listchars test the ':list'.

I wrote a patch.
Please include this.

Thanks.
--
Best regards,
Hirohito Higashi (a.k.a. h_east)

-- 
-- 
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 [email protected].
For more options, visit https://groups.google.com/d/optout.
diff -r 3790fb70e04c src/message.c
--- a/src/message.c	Tue Apr 21 19:10:49 2015 +0200
+++ b/src/message.c	Wed Apr 22 12:29:28 2015 +0900
@@ -1761,6 +1761,11 @@
 		c = lcs_trail;
 		attr = hl_attr(HLF_8);
 	    }
+	    else if (c == ' ' && list && lcs_space != NUL)
+	    {
+		c = lcs_space;
+		attr = hl_attr(HLF_8);
+	    }
 	}
 
 	if (c == NUL)

Raspunde prin e-mail lui