unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* font-lock question
@ 2008-07-15 12:19 Klaus Zeitler
  2008-07-24 15:11 ` Stefan Monnier
  0 siblings, 1 reply; 3+ messages in thread
From: Klaus Zeitler @ 2008-07-15 12:19 UTC (permalink / raw)
  To: emacs-devel

For many years I had the following statement in my .emacs file:

(defun show-tabs () "Show tabs with a slightly changed background"
  (font-lock-add-keywords nil '(("\t"  (0 'tab-face t))) t))
(add-hook 'font-lock-mode-hook 'show-tabs)

This has worked without any problems up to 22.1.
In version 22.2 and 23 I do have problems with syntax highlighting due
to these few lines above, e.g. when I try M-x list-faces-display
all colors are gone.

So here's my question. Did that only work by accident in older versions?
Maybe I have to change this code slightly for 22.2 and CVS version, but I
can't see what's wrong, when I look at the docu for font-lock-add-keywords.

Klaus

-- 
 --------------------------------------------------
|  Klaus Zeitler      Alcatel-Lucent               |
|  Email:             kzeitler@alcatel-lucent.com  |
 --------------------------------------------------
---
Copy Protection:  A clever method of preventing incompetent pirates from
stealing software and legitimate customers from using it.




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: font-lock question
  2008-07-15 12:19 font-lock question Klaus Zeitler
@ 2008-07-24 15:11 ` Stefan Monnier
  2008-07-25 13:33   ` Klaus Zeitler
  0 siblings, 1 reply; 3+ messages in thread
From: Stefan Monnier @ 2008-07-24 15:11 UTC (permalink / raw)
  To: kzeitler; +Cc: emacs-devel

> For many years I had the following statement in my .emacs file:
> (defun show-tabs () "Show tabs with a slightly changed background"
>   (font-lock-add-keywords nil '(("\t"  (0 'tab-face t))) t))
> (add-hook 'font-lock-mode-hook 'show-tabs)

> This has worked without any problems up to 22.1.
> In version 22.2 and 23 I do have problems with syntax highlighting due
> to these few lines above, e.g. when I try M-x list-faces-display
> all colors are gone.

> So here's my question. Did that only work by accident in older versions?
> Maybe I have to change this code slightly for 22.2 and CVS version, but I
> can't see what's wrong, when I look at the docu for font-lock-add-keywords.

Yes, looks like a bug to me,


        Stefan




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: font-lock question
  2008-07-24 15:11 ` Stefan Monnier
@ 2008-07-25 13:33   ` Klaus Zeitler
  0 siblings, 0 replies; 3+ messages in thread
From: Klaus Zeitler @ 2008-07-25 13:33 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

>>>>> "Stefan" == Stefan Monnier <monnier@IRO.UMontreal.CA> writes:
    Stefan> 
    >> So here's my question. Did that only work by accident in older versions?
    >> Maybe I have to change this code slightly for 22.2 and CVS version, but I
    >> can't see what's wrong, when I look at the docu for font-lock-add-keywords.
    Stefan> 
    Stefan> Yes, looks like a bug to me,

I'll file a bug report next week.

Klaus


-- 
 --------------------------------------------------
|  Klaus Zeitler      Alcatel-Lucent               |
|  Email:             kzeitler@alcatel-lucent.com  |
 --------------------------------------------------
---
Never let your sense of morals prevent you from doing
what is right.         -- Salvor Hardin, "Foundation"




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-07-25 13:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-07-15 12:19 font-lock question Klaus Zeitler
2008-07-24 15:11 ` Stefan Monnier
2008-07-25 13:33   ` Klaus Zeitler

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).