unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Juri Linkov <juri@jurta.org>
Subject: Re: ChangeLog fontifications
Date: Tue, 11 May 2004 21:03:03 +0300	[thread overview]
Message-ID: <874qqmdep6.fsf@mail.jurta.org> (raw)
In-Reply-To: <ulljyrik7.fsf@gnu.org> (Sam Steingold's message of "Tue, 11 May 2004 13:15:52 -0400")

Sam Steingold <sds@gnu.org> writes:
> when ChangeLog records a change in a Common Lisp function with a
> package prefix, only the package name is highlighted, not the function
> name:
>
> 	* foo.lisp (PACK:FUNCTION-NAME): optimized frobnifications
>
> "PACK" is highlighted but "FUNCTION-NAME" is not.
>
> I am pretty sure that `change-log-font-lock-keywords' in add-log.el
> needs to be fixed, but I am not quite sure how...

I see no harm in removing : from regexps of function names.  I don't
understand why such restriction was added here, given the fact that
parentheses around a function name already pretty safely enclose it.

Index: lisp/add-log.el
===================================================================
RCS file: /cvsroot/emacs/emacs/lisp/add-log.el,v
retrieving revision 1.149
diff -u -r1.149 add-log.el
--- lisp/add-log.el	19 Apr 2004 11:31:05 -0000	1.149
+++ lisp/add-log.el	11 May 2004 17:57:14 -0000
@@ -230,13 +230,13 @@
      ;; Possibly further names in a list:
      ("\\=, \\([^ ,:([\n]+\\)" nil nil (1 'change-log-file-face))
      ;; Possibly a parenthesized list of names:
-     ("\\= (\\([^) ,:\n]+\\)" nil nil (1 'change-log-list-face))
-     ("\\=, *\\([^) ,:\n]+\\)" nil nil (1 'change-log-list-face)))
+     ("\\= (\\([^) ,\n]+\\)" nil nil (1 'change-log-list-face))
+     ("\\=, *\\([^) ,\n]+\\)" nil nil (1 'change-log-list-face)))
     ;;
     ;; Function or variable names.
-    ("^\t(\\([^) ,:\n]+\\)"
+    ("^\t(\\([^) ,\n]+\\)"
      (1 'change-log-list-face)
-     ("\\=, *\\([^) ,:\n]+\\)" nil nil (1 'change-log-list-face)))
+     ("\\=, *\\([^) ,\n]+\\)" nil nil (1 'change-log-list-face)))
     ;;
     ;; Conditionals.
     ("\\[!?\\([^]\n]+\\)\\]\\(:\\| (\\)" (1 'change-log-conditionals-face))

-- 
Juri Linkov
http://www.jurta.org/emacs/

  reply	other threads:[~2004-05-11 18:03 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-05-11 17:15 ChangeLog fontifications Sam Steingold
2004-05-11 18:03 ` Juri Linkov [this message]
2004-05-11 22:54   ` Sam Steingold
2004-05-11 23:02     ` Miles Bader
2004-05-12  8:33       ` Werner LEMBERG
2004-05-12 10:10         ` Miles Bader
2004-05-12 12:36           ` Eli Zaretskii
2004-05-13 17:22           ` Sam Steingold
2004-05-14 21:01             ` Richard Stallman
2004-05-11 23:26     ` Stefan Monnier
2004-05-12  0:08       ` Juri Linkov
2004-05-12  5:48         ` Eli Zaretskii
2004-05-12 12:55           ` Stefan Monnier
  -- strict thread matches above, loose matches on Subject: below --
2004-05-13 18:50 Bruno Haible
2004-05-13 19:26 ` Miles Bader
2004-05-13 19:43   ` Bruno Haible
2004-05-13 20:38     ` Miles Bader
2004-05-13 21:12       ` Sam Steingold
2004-05-13 21:15         ` Miles Bader
2004-05-13 20:13 ` Alan Shutko
2004-05-13 20:42   ` Miles Bader
2004-05-13 21:37   ` Werner LEMBERG
2004-05-13 21:56     ` Miles Bader
2004-05-13 22:25       ` Stefan Monnier
2004-05-14  3:00       ` Karl Eichwalder
2004-05-14  4:24         ` Miles Bader
2004-05-14 13:56         ` Robert J. Chassell
2004-05-14  6:24       ` Werner LEMBERG
2004-05-14  6:36         ` Werner LEMBERG
2004-05-14 15:31       ` Sam Steingold
2004-05-14 21:02 ` Richard Stallman
2004-05-16 15:14   ` Bruno Haible
2004-05-16 16:52     ` Eli Zaretskii
2004-05-16 20:52       ` Miles Bader
2004-05-17 22:57     ` Richard Stallman
2004-05-18  5:02       ` Karl Eichwalder
2004-05-19  1:32         ` Miles Bader
2004-05-19 13:08           ` Sam Steingold
2004-05-19 13:22             ` Miles Bader
2004-05-19 14:59               ` Stefan Monnier
2004-05-19 13:45         ` Richard Stallman

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

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=874qqmdep6.fsf@mail.jurta.org \
    --to=juri@jurta.org \
    /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 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).