bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#4928: marked as done (23.1; lm-keywords more flexibility)


From: Emacs bug Tracking System
Subject: bug#4928: marked as done (23.1; lm-keywords more flexibility)
Date: Tue, 17 Nov 2009 00:30:06 +0000

Your message dated Tue, 17 Nov 2009 11:21:40 +1100
with message-id <address@hidden>
and subject line Re: bug#4928: 23.1; lm-keywords more flexibility
has caused the Emacs bug report #4928,
regarding 23.1; lm-keywords more flexibility
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact address@hidden
immediately.)


-- 
4928: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=4928
Emacs Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: 23.1; lm-keywords more flexibility Date: Sun, 15 Nov 2009 11:30:13 +1100 User-agent: Gnus/5.110011 (No Gnus v0.11) Emacs/23.1 (gnu/linux)
This is a idea to relax/extend lm-keywords to cope with multi-line
keywords, like say Drew's icicles.el,

  ;; Keywords: internal, extensions, help, abbrev, local, minibuffer,
  ;;           keys, apropos, completion, matching, regexp, command

And comma-only separators like say zenirc-color.el,

  ;; Keywords: zenirc,extensions

And trailing-space as in say vm-summary-faces.el, which had been ok in
Emacs 21.

  ;; Keywords:    VM 


2009-11-13  Kevin Ryde  <address@hidden>

        * emacs-lisp/lisp-mnt.el (lm-keywords): Allow multi-line keywords.
        (lm-keywords-list): Allow comma-only separator like "foo,bar".
        Ignore trailing spaces by omit-nulls to split-string (fixing
        regression from Emacs 21 due to the incompatible split-string
        change.)

--- lisp-mnt.el.~1.63.~ 2009-10-08 11:00:31.000000000 +1100
+++ lisp-mnt.el 2009-11-15 11:27:05.000000000 +1100
@@ -445,16 +445,20 @@
            (match-string-no-properties 1)))))))
 
 (defun lm-keywords (&optional file)
-  "Return the keywords given in file FILE, or current buffer if FILE is nil."
+  "Return the keywords given in file FILE, or current buffer if FILE is nil.
+The return value is a `downcase'-ed string, or nil if no keywords
+header.  Multi-line keywords are joined up with a space between
+each line."
   (lm-with-file file
-    (let ((keywords (lm-header "keywords")))
-      (and keywords (downcase keywords)))))
+    (let ((keywords (lm-header-multiline "keywords")))
+      (and keywords
+          (mapconcat 'downcase keywords " ")))))
 
 (defun lm-keywords-list (&optional file)
   "Return list of keywords given in file FILE."
   (let ((keywords (lm-keywords file)))
     (if keywords
-       (split-string keywords ",?[ \t]"))))
+       (split-string keywords "[, \t\n]+" t))))
 
 (defvar finder-known-keywords)
 (defun lm-keywords-finder-p (&optional file)

In GNU Emacs 23.1.1 (i486-pc-linux-gnu, GTK+ Version 2.16.5)
 of 2009-09-14 on raven, modified by Debian
configured using `configure  '--build=i486-linux-gnu' '--host=i486-linux-gnu' 
'--prefix=/usr' '--sharedstatedir=/var/lib' '--libexecdir=/usr/lib' 
'--localstatedir=/var/lib' '--infodir=/usr/share/info' 
'--mandir=/usr/share/man' '--with-pop=yes' 
'--enable-locallisppath=/etc/emacs23:/etc/emacs:/usr/local/share/emacs/23.1/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/23.1/site-lisp:/usr/share/emacs/site-lisp:/usr/share/emacs/23.1/leim'
 '--with-x=yes' '--with-x-toolkit=gtk' '--with-toolkit-scroll-bars' 
'build_alias=i486-linux-gnu' 'host_alias=i486-linux-gnu' 'CFLAGS=-DDEBIAN -g 
-O2' 'LDFLAGS=-g' 'CPPFLAGS=''

--- End Message ---
--- Begin Message --- Subject: Re: bug#4928: 23.1; lm-keywords more flexibility Date: Tue, 17 Nov 2009 11:21:40 +1100 User-agent: Gnus/5.110011 (No Gnus v0.11) Emacs/23.1 (gnu/linux)
Stefan Monnier <address@hidden> writes:
>
> Feel free to install it,

Done.

--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]