[PATCH v2 3/7] emacs: rename `notmuch-show-toggle-headers' to `notmuch-show-toggle-visibility-headers'

Ethan Glasser-Camp ethan.glasser.camp at gmail.com
Fri Oct 12 14:36:31 PDT 2012


Pieter Praet <pieter at praet.org> writes:

> * emacs/notmuch-show.el
>
>   (notmuch-show-toggle-headers):
>     Rename to `notmuch-show-toggle-visibility-headers'.

This patch, and its predecessors, all look great to me. The following
patches were already marked "stale" (and indeed they don't apply). But
it looks like David Edmonson gave his +1 on the idea, so maybe you
should rebase them.

Side note: how do you write these tests? How do you generate what the
output should look like? Copy/paste from a "real" emacs session? You
have generated an awful lot of tests :)

Ethan


More information about the notmuch mailing list