diff options
| author | Jesse Rosenthal <jrosenthal@jhu.edu> | 2010-04-13 14:47:19 -0400 |
|---|---|---|
| committer | Carl Worth <cworth@cworth.org> | 2010-04-14 10:10:55 -0700 |
| commit | db96d67ba2e2ae01bc3139a3a22cc5433880cbae (patch) | |
| tree | b124fa2961bc0282190fcdf9c7798d1c494ba189 /emacs | |
| parent | 2fbb6d05a931b23c307aee3af05ab488c6bca425 (diff) | |
Clean up code duplication in adding or removing tag by region.
Clean up code duplication, as per Carl's suggestion, by making
notmuch-search-{add/remove}-tag-thread a special case of the -region
commands, where the region in question is between (point) and (point).
Diffstat (limited to 'emacs')
| -rw-r--r-- | emacs/notmuch.el | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/emacs/notmuch.el b/emacs/notmuch.el index 178dea2..be09f42 100644 --- a/emacs/notmuch.el +++ b/emacs/notmuch.el @@ -507,8 +507,7 @@ and will also appear in a buffer named \"*Notmuch errors*\"." output))) (defun notmuch-search-add-tag-thread (tag) - (notmuch-call-notmuch-process "tag" (concat "+" tag) (notmuch-search-find-thread-id)) - (notmuch-search-set-tags (delete-dups (sort (cons tag (notmuch-search-get-tags)) 'string<)))) + (notmuch-search-add-tag-region tag (point) (point))) (defun notmuch-search-add-tag-region (tag beg end) (let ((search-id-string (mapconcat 'identity (notmuch-search-find-thread-id-region beg end) " or "))) @@ -522,8 +521,7 @@ and will also appear in a buffer named \"*Notmuch errors*\"." (forward-line)))))) (defun notmuch-search-remove-tag-thread (tag) - (notmuch-call-notmuch-process "tag" (concat "-" tag) (notmuch-search-find-thread-id)) - (notmuch-search-set-tags (delete tag (notmuch-search-get-tags)))) + (notmuch-search-remove-tag-region tag (point) (point))) (defun notmuch-search-remove-tag-region (tag beg end) (let ((search-id-string (mapconcat 'identity (notmuch-search-find-thread-id-region beg end) " or "))) |