[PATCH 1/2] emacs: Add notmuch-combine-face-text-property-string

Austin Clements amdragon at MIT.EDU
Sun Mar 24 19:15:51 PDT 2013


Quoth Tomi Ollila on Mar 23 at  1:42 pm:
> On Sat, Mar 23 2013, Damien Cassou <damien.cassou at gmail.com> wrote:
> 
> > Signed-off-by: Damien Cassou <damien.cassou at gmail.com>
> > ---
> 
> The function notmuch-combine-face-text-property-string is used only
> once in next patch. Do you anticipate more uses of it in the future?

It seems like a logical helper to notmuch-combine-face-text-property,
even if it's currently used in only one place.

> Tomi
> 
> >  emacs/notmuch-lib.el |    8 ++++++++
> >  1 file changed, 8 insertions(+)
> >
> > diff --git a/emacs/notmuch-lib.el b/emacs/notmuch-lib.el
> > index e6edb9a..78c578f 100644
> > --- a/emacs/notmuch-lib.el
> > +++ b/emacs/notmuch-lib.el
> > @@ -357,6 +357,14 @@ OBJECT."
> >  	(setq pos next))))
> >    object)
> >  
> > +(defun notmuch-combine-face-text-property-string (string face &optional below)
> > +  (notmuch-combine-face-text-property
> > +   0
> > +   (length string)
> > +   face
> > +   below
> > +   string))
> > +
> >  (defun notmuch-logged-error (msg &optional extra)
> >    "Log MSG and EXTRA to *Notmuch errors* and signal MSG.
> >  
> >
> > _______________________________________________
> > notmuch mailing list
> > notmuch at notmuchmail.org
> > http://notmuchmail.org/mailman/listinfo/notmuch
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch


More information about the notmuch mailing list