unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [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 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

* [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

* Re: [PATCH 1/1] emacs: Use "message-cited-text" instead of "message-cited-text-face"
  2011-03-20 10:06     ` [PATCH 1/1] " Pieter Praet
@ 2011-05-09  5:10       ` Jameson Graef Rollins
  0 siblings, 0 replies; 5+ messages in thread
From: Jameson Graef Rollins @ 2011-05-09  5:10 UTC (permalink / raw)
  To: Pieter Praet, notmuch

[-- Attachment #1: Type: text/plain, Size: 123 bytes --]

Hi, Pieter.  I applied this patch to the release-candidate/0.6 branch.
Thanks for sending it in, and fixing it up.

jamie.

[-- Attachment #2: Type: application/pgp-signature, Size: 835 bytes --]

^ permalink raw reply	[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).