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