[PATCH] emacs: Remove duplicate declaration of `notmuch-folders'

Carl Worth cworth at cworth.org
Mon Apr 26 10:51:22 PDT 2010


On Mon, 26 Apr 2010 16:09:43 +0100, dme at dme.org wrote:
> From: David Edmondson <dme at dme.org>
[no commit message]

Thanks, David.

Allow me the chance to try to convince you that *every* commit requires
a sentence or two of justification, (beyond the one-line summary just
stating "what" the patch does). There's *always* some utility to saying
"why".

Take this current patch which seems so simple. There were two
declarations, so one had to go. What more could there be to say?

For me, when reviewing this, I wondered "How did there come to be two
different declarations?", and "Is this patch removing the right one?".
So I had to go grep through the source to figure out what had happened.

I could have avoided that if the commit had said something like:

	This variable was moved from notmuch.el to notmuch-lib.el some time
	ago, but the declaration in notmuch.el was left around. Clean
	that up.

[And I just pushed the patch with this added.]

Whether or not that convinces you, perhaps you can humor me and always
try to come up with something to say, even when it seems there's nothing
possible to add. If nothing else, it can be a fun challenge. If you need
more beyond that, you can try writing your commit message as a haiku.

-Carl

-- 
carl.d.worth at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100426/80da418b/attachment.pgp>


More information about the notmuch mailing list