diff options
author | Nicolas Goaziou <n.goaziou@gmail.com> | 2011-02-19 14:03:38 +0000 |
---|---|---|
committer | Bastien Guerry <bzg@altern.org> | 2011-02-26 18:14:38 +0100 |
commit | c2443d48e99aa4c1044dd817b51298586ab197fe (patch) | |
tree | 77827b528da4ea90ba2e70d33780485153fe2737 | |
parent | 67a7ba3ea32d60bd6c4833058ecbaff6476f5c11 (diff) | |
download | org-mode-c2443d48e99aa4c1044dd817b51298586ab197fe.tar.gz |
fix a bug in org-open-at-point
My bad, wrong manipulation. Here is the patch.
To reproduce the error, you can type the following in a fresh Org
#+begin_src org
target some text <<<target>>> another text target
#+end_src
Using C-c C-o on any of the two links will return an error.
Regards,
>From f7738f3e9239fc4fddccc7850dad7a0936087a58 Mon Sep 17 00:00:00 2001
From: Nicolas Goaziou <n.goaziou@gmail.com>
Date: Sat, 19 Feb 2011 09:37:48 +0100
Subject: [PATCH] Fix bug with link to radio target at beginning or end of buffer
* lisp/org.el (org-open-at-point): if a link to a radio target is the
first, (resp. the last), element of a buffer, function cannot find
the property change required to get its boundaries, and
`buffer-substring' is called with an invalid nil argument.
-rw-r--r-- | lisp/org.el | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/lisp/org.el b/lisp/org.el index 008ed8e..03f0b72 100644 --- a/lisp/org.el +++ b/lisp/org.el @@ -9218,8 +9218,10 @@ application the system uses for this file type." pos (if (get-text-property (1+ (point)) 'org-linked-text) (1+ (point)) (point)) path (buffer-substring - (previous-single-property-change pos 'org-linked-text) - (next-single-property-change pos 'org-linked-text))) + (or (previous-single-property-change pos 'org-linked-text) + (point-min)) + (or (next-single-property-change pos 'org-linked-text) + (point-max)))) (throw 'match t)) (save-excursion |