Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el.

Subject: Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el.

Date: Sun, 06 Jul 2014 22:41:02 +0200

To: Karl Fogel

Cc: Tomi Ollila, notmuch@notmuchmail.org

From: Sebastian Lipp

Karl Fogel <kfogel@red-bean.com>
> In general, supplying a log message with the patch with avoid such
> confusion.  If there is some prose expressing what the change is
> supposed to to, and giving any historical context (such as the mailing
> list thread starting from last year), then it will be easy for any
> reviewer to understand what the patch is intended to do, and check if it
> actually does that.

Okay, next time I will provide that information inline. Thought keeping
References and In-Reply-To headers would be sufficient reference.

• Previous message (by thread): Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el.

Thread:

• Karl Fogel—[PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, notmuch::obsolete, notmuch::patch, unread]
  • David Bremner—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
    • Karl Fogel—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
      • Tomi Ollila—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
        • Jani Nikula—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
          • David Bremner—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
            • Jed Brown—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, notmuch::moreinfo, notmuch::patch, unread]
              • Tomi Ollila—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, notmuch::review, unread]
                • David Bremner—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
                  • Tomi Ollila—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
                    • Sebastian Lipp—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
                      • Karl Fogel—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
                        • Sebastian Lipp—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
                          • Karl Fogel—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
                            • Sebastian Lipp—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
                              • Karl Fogel—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
                        • Sebastian Lipp—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
                          • Karl Fogel—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
                          • Tomi Ollila—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
                            • Karl Fogel—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]
                      • Tomi Ollila—Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el. [inbox, unread]