[PATCH 0/2] contrib: pick: remove sync parser

Tomi Ollila tomi.ollila at iki.fi
Mon Aug 26 13:07:16 PDT 2013


On Mon, Aug 26 2013, Mark Walters <markwalters1009 at gmail.com> wrote:

> Pick has two possible parsing methods: a synchronous one (like show)
> and an async one (like search). The async parser has been the default
> since at least the time pick first entered contrib. I don't believe
> there have been any reported problems with it during this time so we
> might as well remove the sync parser and the associated defcustom. We
> also remove the sync tests.

LGTM (and +1 for the code loss :)

>
> Best wishes
>
> Mark

Tomi
>
>
>
>
> Mark Walters (2):
>   contrib: pick: remove sync parser
>   contrib: pick: remove sync tests
>
>  contrib/notmuch-pick/notmuch-pick.el      |   36 +++++-----------
>  contrib/notmuch-pick/run-tests.sh         |    2 +-
>  contrib/notmuch-pick/test/emacs-pick-sync |   64 -----------------------------
>  3 files changed, 12 insertions(+), 90 deletions(-)
>  delete mode 100755 contrib/notmuch-pick/test/emacs-pick-sync
>
> -- 
> 1.7.9.1
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch


More information about the notmuch mailing list