From: Lars Ingebrigtsen <larsi@gnus.org>
To: Lennart Borgman <lennart.borgman@gmail.com>
Cc: 5001@debbugs.gnu.org, erc-discuss@gnu.org
Subject: bug#5001: erc should use 'font-lock-face, not 'face
Date: Thu, 04 Feb 2016 17:56:30 +1100 [thread overview]
Message-ID: <87fux9yndt.fsf__25388.1350778558$1454569049$gmane$org@gnus.org> (raw)
In-Reply-To: <e01d8a50911210356o13fdf394k94653f79f2b32efe@mail.gmail.com> (Lennart Borgman's message of "Sat, 21 Nov 2009 12:56:20 +0100")
Lennart Borgman <lennart.borgman@gmail.com> writes:
> erc does not use the font-lock machinery for coloring the erc buffers.
> If that is good or not I do not know. However there is a problem with
> that it currently use the 'face text property. This prevents hi-lock
> from working in its buffer which would otherwise be useful.
>
> Therefore please replace 'face with 'font-lock-face everywhere in erc.
>
> (I do not send a patch. I think it is just easier to make the change
> which is quite quick to make. I have tested it though.)
It has been suggested that all instances of "'face" should be replaced
with "'font-lock-face" in erc.
I know nothing about font-locking, so I have no idea whether this is a
good idea or not. Are there anybody on the erc mailing list that has on
opinion here?
-*- mode: grep; default-directory: "~/src/emacs/emacs-25/lisp/erc/" -*-
Grep started at Thu Feb 4 17:53:49
grep --color -nH -e "'face" *.el
erc-button.el:79: :type 'face
erc-button.el:84: :type 'face
erc-button.el:92: :type 'face
erc-button.el:393: (let ((old (erc-list (get-text-property from 'face)))
erc-button.el:395: (end (next-single-property-change from 'face nil to))
erc-button.el:403: (put-text-property pos end 'face new)
erc-button.el:405: old (erc-list (get-text-property pos 'face))
erc-button.el:406: end (next-single-property-change pos 'face nil to)))))
erc-capab.el:194: 'face 'erc-capab-identify-unidentified))))))
erc-dcc.el:1208: 'dcc-chat-privmsg ?n (erc-propertize erc-dcc-from 'face
erc-goodies.el:478: 'face
erc-list.el:75: 'face 'fixed-pitch)
erc-list.el:79: 'face 'fixed-pitch)
erc-list.el:157: 'face 'fixed-pitch)
erc-match.el:489: 'face match-face (current-buffer)))
erc-match.el:498: 'face match-face (current-buffer))
erc-match.el:503: 'face match-face))))
erc-match.el:508: 'face match-face (current-buffer)))
erc-match.el:524: 'face face))))
erc-match.el:533: 'face match-face)))
erc-stamp.el:350: (erc-put-text-property 0 (length ts) 'face 'erc-timestamp-face ts)
erc-track.el:812: (put-text-property 0 (length name) 'face faces name))
erc-track.el:890: (let ((choice (catch 'face
erc-track.el:893: (throw 'face candidate)))))
erc-track.el:895: (catch 'face
erc-track.el:898: (throw 'face candidate)))))))
erc-track.el:974: (faces (let ((face1 (get-text-property 0 'face str)))
erc-track.el:977: (while (and (setq i (next-single-property-change i 'face str m))
erc-track.el:979: (and (setq cur (get-text-property i 'face str))
erc-track.el:987: 'face '(bold erc-current-nick-face)
erc.el:2273: 'face 'erc-input-face)))))
erc.el:2463: 'face (or (intern-soft
erc.el:3892: 'face (or face 'erc-prompt-face)
erc.el:4262: (erc-put-text-property 0 (length mark-s) 'face msg-face str)
erc.el:4264: 'face nick-face str)
erc.el:4266: 'face msg-face str)
erc.el:4303: 'face 'erc-nick-prefix-face)
erc.el:4314: (erc-propertize open 'face 'erc-default-face)
erc.el:4315: (erc-propertize mode 'face 'erc-my-nick-prefix-face)
erc.el:4316: (erc-propertize nick 'face 'erc-my-nick-face)
erc.el:4317: (erc-propertize close 'face 'erc-default-face)))
erc.el:4319: (erc-propertize prefix 'face 'erc-default-face))))
erc.el:5240: 'face 'erc-notice-face s)
erc.el:5243: (erc-put-text-property 0 (length s) 'face 'erc-notice-face s)
erc.el:5255: (erc-put-text-property 0 (length s) 'face 'erc-error-face s)
erc.el:5445: 'face 'erc-command-indicator-face)
erc.el:5465: 'face 'erc-input-face))
erc.el:5889: 'face 'erc-command-indicator-face sp)
erc.el:5899: 'face 'erc-input-face line)
erc.el:6381: 'face face)
erc.el:6385: (erc-propertize header 'face face)
Grep finished (matches found) at Thu Feb 4 17:53:49
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
next prev parent reply other threads:[~2016-02-04 6:56 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-21 11:56 bug#5001: erc should use 'font-lock-face, not 'face Lennart Borgman
2015-12-27 22:20 ` Lars Ingebrigtsen
2015-12-28 2:00 ` Lennart Borgman
2016-02-04 6:56 ` Lars Ingebrigtsen [this message]
[not found] ` <87fux9yndt.fsf@gnus.org>
2016-02-04 16:47 ` bug#5001: [Erc-discuss] " Vivek Dasmohapatra
[not found] ` <alpine.DEB.2.02.1602041644310.32219@platypus.pepperfish.net>
2016-02-05 1:32 ` Lars Ingebrigtsen
[not found] ` <87r3gseyc3.fsf@gnus.org>
2016-02-05 2:38 ` Lars Ingebrigtsen
[not found] ` <87io23ev98.fsf@gnus.org>
2016-03-06 18:03 ` Vivek Dasmohapatra
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='87fux9yndt.fsf__25388.1350778558$1454569049$gmane$org@gnus.org' \
--to=larsi@gnus.org \
--cc=5001@debbugs.gnu.org \
--cc=erc-discuss@gnu.org \
--cc=lennart.borgman@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.