all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Aaron Ecay <aaronecay@gmail.com>
To: Bastien <bzg@altern.org>
Cc: 13529-done@debbugs.gnu.org
Subject: bug#13529: 24.3.50; wrong-type-argument with show-paren-mode
Date: Thu, 24 Jan 2013 14:45:51 -0500	[thread overview]
Message-ID: <87k3r2xt3k.fsf@gmail.com> (raw)
In-Reply-To: <87fw1s2odb.fsf@bzg.ath.cx>

Bastien,

Thanks for the fix.  The git checkout I was working from was just a few
commits behind it.

I am however still getting a different error with your patch.  It looks
like the new (if (integerp ...) ...) excluded a couple of overlay-put
calls it should have included.  See if this patch looks good to you
(applied on top of yours).

diff --git i/lisp/paren.el w/lisp/paren.el
index bf2238d..a9d3be6 100644
--- i/lisp/paren.el
+++ w/lisp/paren.el
@@ -259,11 +259,10 @@ matching parenthesis is highlighted in `show-paren-style' after
 			    (- (point) dir)))))
 	      (if show-paren-overlay
 		  (move-overlay show-paren-overlay from to (current-buffer))
-		(setq show-paren-overlay (make-overlay from to nil t)))))
-	  ;;
-	  ;; Always set the overlay face, since it varies.
-	  (overlay-put show-paren-overlay 'priority show-paren-priority)
-	  (overlay-put show-paren-overlay 'face face)))
+		(setq show-paren-overlay (make-overlay from to nil t))))
+            ;; Always set the overlay face, since it varies.
+            (overlay-put show-paren-overlay 'priority show-paren-priority)
+            (overlay-put show-paren-overlay 'face face))))
     ;; show-paren-mode is nil in this buffer.
     (and show-paren-overlay
 	 (delete-overlay show-paren-overlay))

-- 
Aaron Ecay





  reply	other threads:[~2013-01-24 19:45 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-22 23:59 bug#13529: 24.3.50; wrong-type-argument with show-paren-mode Aaron Ecay
2013-01-23 10:19 ` Bastien
2013-01-24 19:45   ` Aaron Ecay [this message]
2013-01-24 21:19     ` Bastien

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87k3r2xt3k.fsf@gmail.com \
    --to=aaronecay@gmail.com \
    --cc=13529-done@debbugs.gnu.org \
    --cc=bzg@altern.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.