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

2012-10-18 Thread Tomi Ollila
On Thu, Oct 18 2012, Ethan wrote: > On Thu, Oct 18, 2012 at 5:50 AM, Tomi Ollila wrote: > >> On Thu, Oct 18 2012, Ethan Glasser-Camp wrote: >> >> > Ethan Glasser-Camp writes: >> > >> >> This patch, and its predecessors, all look great to me. >> > >> > But a note: many of the first lines in your

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

2012-10-18 Thread Tomi Ollila
On Thu, Oct 18 2012, Ethan Glasser-Camp wrote: > Ethan Glasser-Camp writes: > >> This patch, and its predecessors, all look great to me. > > But a note: many of the first lines in your commit messages ("{show, Hmm, first lines -- IIRC mailman adds those tabs to the subject line -- and those are

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

2012-10-18 Thread Tomi Ollila
On Thu, Oct 18 2012, Ethan Glasser-Camp wrote: > Ethan Glasser-Camp writes: > >> This patch, and its predecessors, all look great to me. > > But a note: many of the first lines in your commit messages ("{show, > hide} message headers") contain tabs. I hate tabs. Is this intentional? > I have

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

2012-10-18 Thread Ethan
On Thu, Oct 18, 2012 at 5:50 AM, Tomi Ollila wrote: > On Thu, Oct 18 2012, Ethan Glasser-Camp wrote: > > > Ethan Glasser-Camp writes: > > > >> This patch, and its predecessors, all look great to me. > > > > But a note: many of the first lines in your commit messages ("{show, > > Hmm, first

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

2012-10-18 Thread Tomi Ollila
On Thu, Oct 18 2012, Ethan Glasser-Camp wrote: Ethan Glasser-Camp ethan.glasser.c...@gmail.com writes: This patch, and its predecessors, all look great to me. But a note: many of the first lines in your commit messages ({show, hide} message headers) contain tabs. I hate tabs. Is this

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

2012-10-18 Thread Ethan
On Thu, Oct 18, 2012 at 5:50 AM, Tomi Ollila tomi.oll...@iki.fi wrote: On Thu, Oct 18 2012, Ethan Glasser-Camp wrote: Ethan Glasser-Camp ethan.glasser.c...@gmail.com writes: This patch, and its predecessors, all look great to me. But a note: many of the first lines in your commit

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

2012-10-17 Thread Ethan Glasser-Camp
Ethan Glasser-Camp writes: > This patch, and its predecessors, all look great to me. But a note: many of the first lines in your commit messages ("{show, hide} message headers") contain tabs. I hate tabs. Is this intentional? I have noticed it on other patches you've sent (such as

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

2012-10-17 Thread Ethan Glasser-Camp
Ethan Glasser-Camp ethan.glasser.c...@gmail.com writes: This patch, and its predecessors, all look great to me. But a note: many of the first lines in your commit messages ({show, hide} message headers) contain tabs. I hate tabs. Is this intentional? I have noticed it on other patches you've

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

2012-10-12 Thread Ethan Glasser-Camp
Pieter Praet 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

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

2012-10-12 Thread Ethan Glasser-Camp
Pieter Praet pie...@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

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

2012-02-22 Thread Pieter Praet
* emacs/notmuch-show.el (notmuch-show-toggle-headers): Rename to `notmuch-show-toggle-visibility-headers'. (notmuch-show-mode-map): Update "h" binding wrt renamed `notmuch-show-toggle-headers'. (notmuch-message-headers): Update docstring wrt renamed

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

2012-02-22 Thread Pieter Praet
* emacs/notmuch-show.el (notmuch-show-toggle-headers): Rename to `notmuch-show-toggle-visibility-headers'. (notmuch-show-mode-map): Update h binding wrt renamed `notmuch-show-toggle-headers'. (notmuch-message-headers): Update docstring wrt renamed