Re: [PATCH] test: test-lib.el: replace sleep-for with sit-for in notmuch-test-wait

Subject: Re: [PATCH] test: test-lib.el: replace sleep-for with sit-for in notmuch-test-wait

Date: Sat, 04 Aug 2012 10:05:04 +0300

To: Austin Clements

Cc: notmuch@notmuchmail.org

From: Tomi Ollila

On Fri, Aug 03 2012, Austin Clements <amdragon@MIT.EDU> wrote:

> Code LGTM, but maybe the comment should explain that this is a
> workaround for a bug in Emacs, given that both sleep-for and sit-for
> are supposed to process process output and run sentinels (which has
> been stated in the Elisp reference at least as far back as Emacs 21).
> In light of this (and our better understanding of sleep-for and
> sit-for from IRC discussions), the commit message is also misleading.

Indeed...

Tomi

Thread: