unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Fontification of ChangeLog entries.
@ 2005-04-01 15:19 Lute Kamstra
  2005-04-03  5:20 ` Richard Stallman
  0 siblings, 1 reply; 4+ messages in thread
From: Lute Kamstra @ 2005-04-01 15:19 UTC (permalink / raw)


When a name in a parenthesized list of a ChangeLog entry contains
spaces, only the first word gets fontified.  See {lispref,man}/ChangeLog
for examples.  What about the following patch to fix this?

Lute.


Index: lisp/add-log.el
===================================================================
RCS file: /cvsroot/emacs/emacs/lisp/add-log.el,v
retrieving revision 1.156
diff -u -r1.156 add-log.el
--- lisp/add-log.el	21 Mar 2005 14:37:35 -0000	1.156
+++ lisp/add-log.el	1 Apr 2005 15:16:05 -0000
@@ -1,7 +1,7 @@
 ;;; add-log.el --- change log maintenance commands for Emacs
 
-;; Copyright (C) 1985, 86, 88, 93, 94, 97, 98, 2000, 03, 2004
-;;           Free Software Foundation, Inc.
+;; Copyright (C) 1985, 1986, 1988, 1993, 1994, 1997, 1998, 2000, 2003,
+;;   2004, 2005 Free Software Foundation, Inc.
 
 ;; Maintainer: FSF
 ;; Keywords: tools
@@ -230,15 +230,15 @@
      ;; Possibly further names in a list:
      ("\\=, \\([^ ,:([\n]+\\)" nil nil (1 'change-log-file-face))
      ;; Possibly a parenthesized list of names:
-     ("\\= (\\([^() ,\n]+\\|(\\(setf\\|SETF\\) [^() ,\n]+)\\)"
+     ("\\= (\\([^(),\n]+\\|(\\(setf\\|SETF\\) [^() ,\n]+)\\)"
       nil nil (1 'change-log-list-face))
-     ("\\=, *\\([^() ,\n]+\\|(\\(setf\\|SETF\\) [^() ,\n]+)\\)"
+     ("\\=, *\\([^(),\n]+\\|(\\(setf\\|SETF\\) [^() ,\n]+)\\)"
       nil nil (1 'change-log-list-face)))
     ;;
     ;; Function or variable names.
-    ("^\t(\\([^() ,\n]+\\|(\\(setf\\|SETF\\) [^() ,\n]+)\\)"
+    ("^\t(\\([^(),\n]+\\|(\\(setf\\|SETF\\) [^() ,\n]+)\\)"
      (1 'change-log-list-face)
-     ("\\=, *\\([^() ,\n]+\\|(\\(setf\\|SETF\\) [^() ,\n]+)\\)" nil nil
+     ("\\=, *\\([^(),\n]+\\|(\\(setf\\|SETF\\) [^() ,\n]+)\\)" nil nil
       (1 'change-log-list-face)))
     ;;
     ;; Conditionals.

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

* Re: Fontification of ChangeLog entries.
  2005-04-01 15:19 Fontification of ChangeLog entries Lute Kamstra
@ 2005-04-03  5:20 ` Richard Stallman
  2005-04-03 18:54   ` Lute Kamstra
  0 siblings, 1 reply; 4+ messages in thread
From: Richard Stallman @ 2005-04-03  5:20 UTC (permalink / raw)
  Cc: emacs-devel

    When a name in a parenthesized list of a ChangeLog entry contains
    spaces, only the first word gets fontified.  See {lispref,man}/ChangeLog
    for examples.  What about the following patch to fix this?

I agree with the idea of fontifying more than the first word.  The
main question is whether the change has unwanted side effects in other
cases.  The best way to find that out is by trying it.  If it works fine,
I am in favor of it.

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

* Re: Fontification of ChangeLog entries.
  2005-04-03  5:20 ` Richard Stallman
@ 2005-04-03 18:54   ` Lute Kamstra
  2005-04-04  6:18     ` Richard Stallman
  0 siblings, 1 reply; 4+ messages in thread
From: Lute Kamstra @ 2005-04-03 18:54 UTC (permalink / raw)
  Cc: emacs-devel

Richard Stallman <rms@gnu.org> writes:

>     When a name in a parenthesized list of a ChangeLog entry contains
>     spaces, only the first word gets fontified.  See {lispref,man}/ChangeLog
>     for examples.  What about the following patch to fix this?
>
> I agree with the idea of fontifying more than the first word.  The
> main question is whether the change has unwanted side effects in other
> cases.  The best way to find that out is by trying it.  If it works fine,
> I am in favor of it.

I've browsed a large number of Emacs' ChangeLog files and saw no
problems.  Shall I install the patch so that others will test it too?

Lute.

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

* Re: Fontification of ChangeLog entries.
  2005-04-03 18:54   ` Lute Kamstra
@ 2005-04-04  6:18     ` Richard Stallman
  0 siblings, 0 replies; 4+ messages in thread
From: Richard Stallman @ 2005-04-04  6:18 UTC (permalink / raw)
  Cc: emacs-devel

    I've browsed a large number of Emacs' ChangeLog files and saw no
    problems.  Shall I install the patch so that others will test it too?

Yes, please.

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

end of thread, other threads:[~2005-04-04  6:18 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-04-01 15:19 Fontification of ChangeLog entries Lute Kamstra
2005-04-03  5:20 ` Richard Stallman
2005-04-03 18:54   ` Lute Kamstra
2005-04-04  6:18     ` Richard Stallman

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).