summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEli Zaretskii <eliz@gnu.org>2016-02-27 13:27:36 +0200
committerEli Zaretskii <eliz@gnu.org>2016-02-27 13:27:36 +0200
commit14060a9c1679174b37bba7140c6b715d90502d70 (patch)
tree7dbc80ba9341eaec5c9d4caf7536587daca52f66
parent098d47bcf735a1a194a155a8d365746a364617d8 (diff)
downloademacs-14060a9c1679174b37bba7140c6b715d90502d70.tar.gz
Avoid inflooping in thing-at-point-looking-at
* lisp/thingatpt.el (thing-at-point-looking-at): Avoid inflooping with regular expressions whose matching doesn't move point. (Bug#22756) Describe the argument DISTANCE in the doc string.
-rw-r--r--lisp/thingatpt.el13
1 files changed, 10 insertions, 3 deletions
diff --git a/lisp/thingatpt.el b/lisp/thingatpt.el
index 1686c02ada3..9920fa06d0c 100644
--- a/lisp/thingatpt.el
+++ b/lisp/thingatpt.el
@@ -489,19 +489,26 @@ looks like an email address, \"ftp://\" if it starts with
(defun thing-at-point-looking-at (regexp &optional distance)
"Return non-nil if point is in or just after a match for REGEXP.
Set the match data from the earliest such match ending at or after
-point."
+point.
+
+Optional argument DISTANCE limits search for REGEXP forward and
+back from point."
(save-excursion
(let ((old-point (point))
(forward-bound (and distance (+ (point) distance)))
(backward-bound (and distance (- (point) distance)))
- match)
+ match prev-pos new-pos)
(and (looking-at regexp)
(>= (match-end 0) old-point)
(setq match (point)))
;; Search back repeatedly from end of next match.
;; This may fail if next match ends before this match does.
(re-search-forward regexp forward-bound 'limit)
- (while (and (re-search-backward regexp backward-bound t)
+ (setq prev-pos (point))
+ (while (and (setq new-pos (re-search-backward regexp backward-bound t))
+ ;; Avoid inflooping with some regexps, such as "^",
+ ;; matching which never moves point.
+ (< new-pos prev-pos)
(or (> (match-beginning 0) old-point)
(and (looking-at regexp) ; Extend match-end past search start
(>= (match-end 0) old-point)