Re: removing trailing blanks (and keeping them away)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Feb 04, 2008 at 06:42:01PM +0000, Richard W.M. Jones wrote:
> Jim Meyering wrote:
> >Any objection to a mass removal of trailing blanks?
> >
> >[ If you haven't yet been adversely affected by e.g., pointless conflicts
> >  due solely to differences in white space, count yourself lucky, and
> >  chalk it up to not having to deal with many branches.
> >  The sooner we do this the better, IME. ]
> >
> >I'm proposing two things:
> >  - remove all trailing blanks
> >  - add a "make syntax-check" check to ensure that no new ones are added
> 
> Yup, no problems with such a patch.
> 
> Interestingly emacs perl-mode highlights trailing spaces.
> 
> I looked into it, and there is a show-trailing-whitespace variable in 
> emacs, so one might consider modifying our "standard" trailer to be:
> 
> /*
>  * Local variables:
>  *  indent-tabs-mode: nil
>  *  c-indent-level: 4
>  *  c-basic-offset: 4
>  *  tab-width: 4
>  *  show-trailing-whitespace: t        <--- NB
>  * End:
>  */
> 
> HOWEVER this doesn't work, or at least not automatically.  Emacs 
> complains: "The local variables list in foo.c contains values that may 
> not be safe" and marks show-trailing-whitespace as one such variable. 
> Although one can mark it as safe in your local preferences, every emacs 
> user would have to do this.
> 
> I've no idea why show-trailing-whitespace should be unsafe.

Hmm, I've got the following in my $HOME/.emacsrc file

(require 'show-wspace) 
(add-hook 'font-lock-mode-hook 'highlight-trailing-whitespace)

Which does the trick for me

Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]