notmuch/emacs, branch 0.10

emacs: add notmuch-show-worker function for specifying crypto processing directly

2011-11-13T19:22:16Z

The main reason to introduce this new unexposed function is to allow the buffer redisplay crypto switch to behaving in a more expected way. The prefix to notmuch-show-redisplay buffer now switches the crypto processing of the current show buffer, as opposed to switching the logic of the notmuch-crypto-process-mime customization variable. This behavior is more intuitive.

emacs: add documentation for notmuch-show crypto-switch option

2011-11-13T19:21:58Z

emacs: update notmuch-crypto-process-mime config variable documentation.

2011-11-13T00:50:22Z

This mentions the fact that prefix arguments are now used to enable to crypto switch.

emacs: Unbind M-RET as display of thread with crypto switch.

2011-11-13T00:42:25Z

Use prefix argument instead to set switch.

emacs: add keybind and function to stash Message-ID without prefix

2011-11-13T00:29:04Z

Add function `notmuch-show-stash-message-id-stripped' which stashes a Message-ID after ripping off the prefix and quotes, add bind it to "I" key in `notmuch-show-stash-map'. Simplifying `notmuch-show-get-message-id' instead might seem better, but that would require concat'ing in 9 places instead of 1. Signed-off-by: Pieter Praet

emacs: add invisible space after the search widget field in notmuch-hello

2011-11-12T15:34:53Z

It is very convenient when C-e (bound to `widget-end-of-line') ignores trailing spaces inside the search widget. But it only does so if a widget is not followed by a newline (that is why it works in the saved search widgets). The patch just adds an invisible space after the search widget to get the desirable behavior of `widget-end-of-line'. The extra space is also added to expected results of emacs tests.

emacs: Use a single buffer invisibility spec to fix quadratic search cost.

2011-11-12T14:21:03Z

Buffer redisplay requires traversing the buffer's invisibility spec for every part of the display that has an 'invisible text or overlay property. Previously, the search buffer's invisibility spec list contained roughly one entry for each search result. As a result, redisplay took O(NM) time where N is the number of visible lines and M is the total number of results. On a slow computer, this is enough to make even buffer motion noticeably slow. Worse, during a search operation, redisplay is triggered for each search result (even if there are no visible buffer changes), so search was quadratic (O(NM^2)) in the number of search results. This change switches to using a single element buffer invisibility spec. To un-hide authors, instead of removing an entry from the invisibility spec, it simply removes the invisibility overlay from those authors. I tested using a query with 6633 results on a 9 year old machine. Before this patch, Emacs took 70 seconds to fill the search buffer; toward the end of the search, Emacs consumed 10-20x as much CPU as notmuch; and moving point in the buffer took about a second. With this patch, the same query takes 40 seconds, Emacs consumes ~3x the CPU of notmuch by the end, and there's no noticeable lag to moving point. (There's still some source of non-linearity, because Emacs and notmuch consume roughly the same amount of CPU early in the search.)

Do not query on notmuch-search exit

2011-11-08T18:25:07Z

Emacs 23.2 queries by default about killing existing processes. This is annoying when one wants to interrupt long search with 'q' key. Disable this behavior for notmuch.

emacs: remove unused `point-invisible-p' function

2011-11-08T00:38:37Z

`point-invisible-p' does not work correctly when `invisible' property is a list. There are standard `invisible-p' and related functions that should be used instead.

emacs: remove no longer used functions from notmuch-show.el

2011-11-08T00:38:37Z

Remove `notmuch-show-move-past-invisible-backward' and `notmuch-show-move-past-invisible-forward' functions which are unused.