Re: [PATCH 08/10] test: set variables using `let' instead of `setq' in Emacs tests

Subject: Re: [PATCH 08/10] test: set variables using `let' instead of `setq' in Emacs tests

Date: Mon, 23 Jan 2012 20:05:06 +0100

To: Dmitry Kurochkin, notmuch@notmuchmail.org

Cc:

From: Xavier Maillard


On Tue, 28 Jun 2011 08:45:09 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:
> Using `setq' for setting variables in Emacs tests affect other
> tests that may run in the same Emacs environment.  Currently it
> works because each test is run in a separate Emacs instance.  But
> in the future multiple tests will run in a single Emacs instance.
> The patch changes all variables to use `let', so the scope of the
> change is limited to a single test.

+1 (definetely)

/Xavier

Thread: