GNU bug report logs - #15867
24.3.50; cperl-mode: Fix default value of cperl-invalid-face

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 15867 in the body.
You can then email your comments to 15867 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Jambunathan K <kjambunathan <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 24.3.50; cperl-mode: Fix default value of cperl-invalid-face 
Date: Tue, 12 Nov 2013 10:41:55 +0530

Fix cperl-invalid-face.  It is quite distracting.

Leo, Nathan, RJoe on Emacswiki and myself all seem to say this is a bad
default.  That means 4 upvotes that this is not good default.

ps: Since editing with cperl-mode may introduce leading whitespaces on
empty lines, it might actually be better to introduce a
'cperl-cleanup-trailing-whitespace' (or some such variable) to
compensate for cperl's smartness.


----------------------------------------------------------------

From a private thread.
=====================

Leo Liu <sdl.web <at> gmail.com> writes:

> On 2013-11-11 20:03 +0800, Nathan Trapuzzano wrote:
>> (defalias 'perl-mode 'cperl-mode)
>> (setq cperl-continued-brace-offset -4
>>       cperl-invalid-face 'default
>> )

From: RJH on http://www.emacswiki.org/emacs/CPerlMode
====================================================

First of all, there’s a very peculiar feature which many people want to
disable: trailing whitespace is indicated with an underline (see
discussion below). Consider that whitespace is rarely significant in
perl code, but that underlines often are (e.g. “$_”) and with this
feature an underscore appears every time you hit the space bar, and will
not disappear until you enter another character. Adding this line to
your .emacs works to disable this feature in GnuEmacs 21:

    (setq cperl-invalid-face (quote off)) 

However, in at least GNU Emacs 21.2.1, this has been known to cause
problems with font-locking, in which case:

    (setq cperl-invalid-face nil) 


In GNU Emacs 24.3.50.4 (i686-pc-linux-gnu, GTK+ Version 2.20.1)
 of 2013-11-09 on debian-6.05
Bzr revision: 115051 jan.h.d <at> swipnet.se-20131109111953-cysfi19meuidy96q
Windowing system distributor `The X.Org Foundation', version 11.0.10707000
System Description:	Debian GNU/Linux 6.0.5 (squeeze)

Message #8 received at 15867 <at> debbugs.gnu.org (full text, mbox):

From: Leo Liu <sdl.web <at> gmail.com>
To: Jambunathan K <kjambunathan <at> gmail.com>
Cc: 15867 <at> debbugs.gnu.org
Subject: Re: bug#15867: 24.3.50;
 cperl-mode: Fix default value of cperl-invalid-face
Date: Tue, 12 Nov 2013 17:26:07 +0800

On 2013-11-12 13:11 +0800, Jambunathan K wrote:
> Fix cperl-invalid-face.  It is quite distracting.
>
> Leo, Nathan, RJoe on Emacswiki and myself all seem to say this is a bad
> default.  That means 4 upvotes that this is not good default.
>
> ps: Since editing with cperl-mode may introduce leading whitespaces on
> empty lines, it might actually be better to introduce a
> 'cperl-cleanup-trailing-whitespace' (or some such variable) to
> compensate for cperl's smartness.

The cperl-invalid-face seems overlapping show-trailing-whitespace. If so
I prefer removing it entirely.

Leo

Message #13 received at 15867-done <at> debbugs.gnu.org (full text, mbox):

From: Jambunathan K <kjambunathan <at> gmail.com>
To: 15867-done <at> debbugs.gnu.org
Subject: Re: bug#15867: 24.3.50;
 cperl-mode: Fix default value of cperl-invalid-face
Date: Tue, 01 Jan 2002 05:52:02 +0530

OP here. Closing it.  I have a local fix that works for me.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.