[notmuch] [PATCH -v2] notmuch.el: Support for customizing search result display
cworth at cworth.org
Fri Feb 5 11:55:26 PST 2010
On Wed, 2 Dec 2009 18:19:38 +0530, "Aneesh Kumar K.V" <aneesh.kumar at linux.vnet.ibm.com> wrote:
> From: Aneesh Kumar K.V <aneesh.kumar at gmail.com>
> This patch helps in customizing search result display
> similar to mutt's index_format. The customization is done
> by defining an alist as below
> (setq notmuch-search-result-format '(("date" . "%s ")
> ("authors" . "%-40s ")
> ("subject" . "%s ")
> ("tags" . "(%s)")))
> The supported keywords are date, count, authors, subject and tags.
> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar at gmail.com>
I'm sorry this patch has lingered so long without comment. There's
really only one problem I see with it:
> +(defcustom notmuch-search-result-format nil
> + "Search result formating. Supported fields are
> + date, count, authors, subject, tags
> +ex: (setq notmuch-search-result-format \(\(\"authors\" . \"%-40s\"\)
> + \(\"subject\" . \"%s\"\)\)\)"
> +:type '(alist :key-type (string) :value-type (string))
> +:group 'notmuch)
> - (insert (format "%s %-7s %-40s %s (%s)\n" date count authors subject tags))
> + (if (not notmuch-search-result-format)
> + (progn (insert (format "%s %-7s %-40s %s" date count authors subject))
> + ;; insert the fontified tag
> + (insert-tags (format "%s" tags))
> + (insert "\n"))
> + (notmuch-search-show-result date count authors subject tags))
I don't like that the new format variable is nil by default and then
there's an open-coded implementation of the default formatting. This has
a couple of problems:
1. The new code is not being exercised by default, so it would be easy
to break it without realizing.
2. The system is not very self-documenting.
If a new user wants to tweak the default format, that will be a lot
easier if they find a customizable variable that describes the
current format. *That* will be a lot easier to modify rather than
trying to learn what should be used instead of "nil".
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: not available
More information about the notmuch