On Tue, Aug 27 2013, Mark Walters <[email protected]> wrote: > v2 of this series is at > id:[email protected] > > v2 had a bug on refresh view (which I should have tested more). The > main pick view only worked by fluke as the initial call to pick-worker > was inside a let binding. > > This version fixes the bug, moves the call to pick-worker outside the > let binding (so the bug would have shown up sooner) and also adds a > test for refresh view. > > It is also rebased on top of > id:[email protected] so that > series should be applied first.
Looks Good, tests pass. +1 > > Best wishes > > Mark Tomi > > > Mark Walters (3): > contrib: pick: remove unneeded variable notmuch-pick-buffer-name > contrib: pick: add docstring for notmuch-pick-worker > contrib: pick: test: refresh view > > contrib/notmuch-pick/notmuch-pick.el | 29 +++++++++++++---------------- > contrib/notmuch-pick/test/emacs-pick | 11 +++++++++++ > 2 files changed, 24 insertions(+), 16 deletions(-) > > -- > 1.7.9.1 > > _______________________________________________ > notmuch mailing list > [email protected] > http://notmuchmail.org/mailman/listinfo/notmuch > -- uussigu _______________________________________________ notmuch mailing list [email protected] http://notmuchmail.org/mailman/listinfo/notmuch
