Re: [O] [PATCH] org.el (org-offer-links-in-entry): Reuse global variable

2014-05-20 Thread Bastien
Hi Albert,

Albert Krewinkel tar...@moltkeplatz.de writes:

 * org.el (org-offer-links-in-entry): Use global variable
   `org-any-link-re' instead of defining a string-equal local variable.

 The `re' variable defined in function `org-offer-links-in-entry' is
 string-equal to `org-any-link-re' and is hence replaced by the latter.

 TINYCHANGE

Finally applied, thanks!

-- 
 Bastien



[O] [PATCH] org.el (org-offer-links-in-entry): Reuse global variable

2014-05-11 Thread Albert Krewinkel
* org.el (org-offer-links-in-entry): Use global variable
  `org-any-link-re' instead of defining a string-equal local variable.

The `re' variable defined in function `org-offer-links-in-entry' is
string-equal to `org-any-link-re' and is hence replaced by the latter.

TINYCHANGE
---
 lisp/org.el | 7 ++-
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/lisp/org.el b/lisp/org.el
index c16fab0..40d6972 100644
--- a/lisp/org.el
+++ b/lisp/org.el
@@ -10709,10 +10709,7 @@ there is one, return it.
   (save-restriction
(widen)
(goto-char marker)
-   (let ((re (concat \\( org-bracket-link-regexp \\)\\|
- \\( org-angle-link-re \\)\\|
- \\( org-plain-link-re \\)))
- (cnt ?0)
+   (let ((cnt ?0)
  (in-emacs (if (integerp nth) nil nth))
  have-zero end links link c)
  (when (and (stringp zero) (string-match org-bracket-link-regexp zero))
@@ -10721,7 +10718,7 @@ there is one, return it.
  (save-excursion
(org-back-to-heading t)
(setq end (save-excursion (outline-next-heading) (point)))
-   (while (re-search-forward re end t)
+   (while (re-search-forward org-any-link-re end t)
  (push (match-string 0) links))
(setq links (org-uniquify (reverse links
  (cond
-- 
2.0.0.rc0