all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Juanma Barranquero <lekktu@gmail.com>
To: Harald Hanche-Olsen <hanche@math.ntnu.no>
Cc: 16799@debbugs.gnu.org
Subject: bug#16799: 24.3.50; delete-selection-mode and electric-pair-mode interaction
Date: Wed, 19 Feb 2014 23:23:40 +0100	[thread overview]
Message-ID: <CAAeL0STjUu=g2CYJdZZgwdHF5Sf2FB1x5kDL5QXOM1XVP028VA@mail.gmail.com> (raw)
In-Reply-To: <20140218.214714.1545557468915065071.hanche@math.ntnu.no>

On Tue, Feb 18, 2014 at 9:47 PM, Harald Hanche-Olsen
<hanche@math.ntnu.no> wrote:

> Error in pre-command-hook (delete-selection-pre-hook): (args-out-of-range 0 1)

I think something like this is needed to avoid
electric-pair-syntax-info parsing before the start of the
region/buffer.

=== modified file 'lisp/elec-pair.el'
--- lisp/elec-pair.el   2014-02-03 00:28:57 +0000
+++ lisp/elec-pair.el   2014-02-19 22:18:02 +0000
@@ -215,8 +215,10 @@
 `electric-pair-pairs' or `electric-pair-text-pairs' were used to
 lookup syntax.  STRING-OR-COMMENT-START indicates that point is
 inside a comment of string."
-  (let* ((pre-string-or-comment (nth 8 (save-excursion
-                                         (syntax-ppss (1- (point))))))
+  (let* ((pre-string-or-comment (if (> (point) (point-min))
+                                   (nth 8 (save-excursion
+                                            (syntax-ppss (1- (point)))))
+                                 t))
          (post-string-or-comment (nth 8 (syntax-ppss (point))))
          (string-or-comment (and post-string-or-comment
                                  pre-string-or-comment))





  reply	other threads:[~2014-02-19 22:23 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-18 20:47 bug#16799: 24.3.50; delete-selection-mode and electric-pair-mode interaction Harald Hanche-Olsen
2014-02-19 22:23 ` Juanma Barranquero [this message]
2014-02-20  4:49   ` Stefan Monnier
2014-02-20 10:38     ` Juanma Barranquero

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='CAAeL0STjUu=g2CYJdZZgwdHF5Sf2FB1x5kDL5QXOM1XVP028VA@mail.gmail.com' \
    --to=lekktu@gmail.com \
    --cc=16799@debbugs.gnu.org \
    --cc=hanche@math.ntnu.no \
    /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.