[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