* [PATCH] emacs: "message-cited-text-face" is obsolete since Emacs 22.1. Use "message-cited-text" instead.
@ 2011-03-16 7:09 Pieter Praet
2011-03-16 11:20 ` David Bremner
0 siblings, 1 reply; 5+ messages in thread
From: Pieter Praet @ 2011-03-16 7:09 UTC (permalink / raw)
To: notmuch
---
emacs/notmuch-wash.el | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/emacs/notmuch-wash.el b/emacs/notmuch-wash.el
index c4a7a41..0e64eb2 100644
--- a/emacs/notmuch-wash.el
+++ b/emacs/notmuch-wash.el
@@ -125,7 +125,7 @@ is what to put on the button."
(let* ((cite-start (match-beginning 0))
(cite-end (match-end 0))
(cite-lines (count-lines cite-start cite-end)))
- (overlay-put (make-overlay cite-start cite-end) 'face 'message-cited-text-face)
+ (overlay-put (make-overlay cite-start cite-end) 'face 'message-cited-text)
(when (> cite-lines (+ notmuch-wash-citation-lines-prefix
notmuch-wash-citation-lines-suffix
1))
@@ -151,7 +151,7 @@ is what to put on the button."
(sig-end-marker (make-marker)))
(set-marker sig-start-marker sig-start)
(set-marker sig-end-marker (point-max))
- (overlay-put (make-overlay sig-start-marker sig-end-marker) 'face 'message-cited-text-face)
+ (overlay-put (make-overlay sig-start-marker sig-end-marker) 'face 'message-cited-text)
(notmuch-wash-region-to-button
sig-start-marker sig-end-marker
"signature" "\n"
---
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] emacs: "message-cited-text-face" is obsolete since Emacs 22.1. Use "message-cited-text" instead.
2011-03-16 7:09 [PATCH] emacs: "message-cited-text-face" is obsolete since Emacs 22.1. Use "message-cited-text" instead Pieter Praet
@ 2011-03-16 11:20 ` David Bremner
2011-03-20 10:11 ` [PATCH 0/1] emacs: Use "message-cited-text" instead of "message-cited-text-face" Pieter Praet
0 siblings, 1 reply; 5+ messages in thread
From: David Bremner @ 2011-03-16 11:20 UTC (permalink / raw)
To: Pieter Praet, notmuch
Hi Pieter;
Thanks a lot for the the patch. I notice there is no commit message in
the body, so I suspect the chances of Carl applying it as-is are pretty
remote. Of course I could be off-base, but we've had many patches in the
past go back for improved commit messages.
All the best,
David
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 0/1] emacs: Use "message-cited-text" instead of "message-cited-text-face"
2011-03-16 11:20 ` David Bremner
@ 2011-03-20 10:11 ` Pieter Praet
2011-03-20 10:06 ` [PATCH 1/1] " Pieter Praet
0 siblings, 1 reply; 5+ messages in thread
From: Pieter Praet @ 2011-03-20 10:11 UTC (permalink / raw)
To: notmuch
On Wed, 16 Mar 2011 08:20:24 -0300, David Bremner <bremner@unb.ca> wrote:
> [...] I notice there is no commit message in the body, so I suspect
> the chances of Carl applying it as-is are pretty remote. [...]
David,
Thanks for the heads up!
Guess I should've read the patch submission guidelines first...
Peace
-Pieter
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/1] emacs: Use "message-cited-text" instead of "message-cited-text-face"
2011-03-20 10:11 ` [PATCH 0/1] emacs: Use "message-cited-text" instead of "message-cited-text-face" Pieter Praet
@ 2011-03-20 10:06 ` Pieter Praet
2011-05-09 5:10 ` Jameson Graef Rollins
0 siblings, 1 reply; 5+ messages in thread
From: Pieter Praet @ 2011-03-20 10:06 UTC (permalink / raw)
To: notmuch
(describe-face 'message-cited-text-face)
> message-cited-text-face is an alias for the face `message-cited-text'.
> This face is obsolete since 22.1; use `message-cited-text' instead.
Signed-off-by: Pieter Praet <pieter@praet.org>
---
emacs/notmuch-wash.el | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/emacs/notmuch-wash.el b/emacs/notmuch-wash.el
index c4a7a41..0e64eb2 100644
--- a/emacs/notmuch-wash.el
+++ b/emacs/notmuch-wash.el
@@ -125,7 +125,7 @@ is what to put on the button."
(let* ((cite-start (match-beginning 0))
(cite-end (match-end 0))
(cite-lines (count-lines cite-start cite-end)))
- (overlay-put (make-overlay cite-start cite-end) 'face 'message-cited-text-face)
+ (overlay-put (make-overlay cite-start cite-end) 'face 'message-cited-text)
(when (> cite-lines (+ notmuch-wash-citation-lines-prefix
notmuch-wash-citation-lines-suffix
1))
@@ -151,7 +151,7 @@ is what to put on the button."
(sig-end-marker (make-marker)))
(set-marker sig-start-marker sig-start)
(set-marker sig-end-marker (point-max))
- (overlay-put (make-overlay sig-start-marker sig-end-marker) 'face 'message-cited-text-face)
+ (overlay-put (make-overlay sig-start-marker sig-end-marker) 'face 'message-cited-text)
(notmuch-wash-region-to-button
sig-start-marker sig-end-marker
"signature" "\n"
--
1.7.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2011-05-09 5:11 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-03-16 7:09 [PATCH] emacs: "message-cited-text-face" is obsolete since Emacs 22.1. Use "message-cited-text" instead Pieter Praet
2011-03-16 11:20 ` David Bremner
2011-03-20 10:11 ` [PATCH 0/1] emacs: Use "message-cited-text" instead of "message-cited-text-face" Pieter Praet
2011-03-20 10:06 ` [PATCH 1/1] " Pieter Praet
2011-05-09 5:10 ` Jameson Graef Rollins
Code repositories for project(s) associated with this public inbox
https://yhetil.org/notmuch.git/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).