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

[Emacs bug Tracking System]

Your message dated Tue, 17 Nov 2009 11:21:40 +1100
with message-id <87skce2e7f.fsf@blah.blah>
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 owner@emacsbugs.donarmstrong.com
immediately.)


-- 
4928: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=4928
Emacs Bug Tracking System
Contact owner@emacsbugs.donarmstrong.com 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  <user42@zip.com.au>
>
>         * 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 <monnier@iro.umontreal.ca> writes:
> >
> > Feel free to install it,
>
> Done.
>
> --- End Message ---