Re: [PATCH v2] emacs: cl-letf enriched-decode-display-prop for text/encriched display

Subject: Re: [PATCH v2] emacs: cl-letf enriched-decode-display-prop for text/encriched display

Date: Wed, 20 Sep 2017 08:12:21 -0300

To: Tomi Ollila, notmuch@notmuchmail.org

Cc: tomi.ollila@iki.fi

From: David Bremner


Tomi Ollila <tomi.ollila@iki.fi> writes:

> Dynamically bind enriched-decode-display-prop when inserting
> text/enriched part. This complements commit 9b0582383833 for
> emacs versions before 24.4 which do not have advice-add functionality.
>
> Note the (require 'enriched). Without that if 'enriched were not
> already provided, the part processing via mm-display-part would
> eventually load that, providing enriched-decode-display-prop
> overriding our temporary definition (for the first time
> text/enriched part is handled).
> ---

What about a test to show that extending support to older versions of
emacs doesn't cause a regression?

d
_______________________________________________
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch

Thread: