[PATCH] emacs: add completion to "tag all" operation ("*" binding)
Dmitry Kurochkin
dmitry.kurochkin at gmail.com
Thu Jan 26 09:39:22 PST 2012
On Thu, 26 Jan 2012 08:46:00 +0000, David Edmondson <dme at dme.org> wrote:
> On Thu, 26 Jan 2012 05:47:07 +0400, Dmitry Kurochkin <dmitry.kurochkin at gmail.com> wrote:
> > > > + ;; Perform some validation
> > > > + (let ((words action))
> > > > + (when (null words) (error "No operation given"))
> > > > + (while words
> > > > + (unless (string-match-p "^[-+][-+_.[:word:]]+$" (car words))
> > > > + (error "Action must be of the form `+thistag -that_tag'"))
> > > > + (setq words (cdr words))))
> > >
> > > This should really be a mapc or a dolist, but maybe that's for another
> > > patch.
> >
> > Yes, I changed "this_tag" spelling in v2, but would prefer to leave
> > non-trivial changes in this code for another patch.
>
> If you were going to submit another patch then fine, but the chances are
> it will just get forgotten.
No worries, v5 has a cleanup patch added :)
Regards,
Dmitry
More information about the notmuch
mailing list