From: Eli Zaretskii <eliz@gnu.org>
Date: Fri, 22 Dec 2006 22:57:15 +0000 (+0000)
Subject: (outline-next-visible-heading): Fix the case with a header at end-of-file
X-Git-Tag: emacs-pretest-22.0.93~487
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=cf27cc8311fea87cb0c4181b47e2cf88929a8de2;p=emacs.git

(outline-next-visible-heading): Fix the case with a header at end-of-file
with no final newline.
---

diff --git a/lisp/outline.el b/lisp/outline.el
index b44fd288bfc..95594ebec9f 100644
--- a/lisp/outline.el
+++ b/lisp/outline.el
@@ -652,19 +652,24 @@ A heading line is one that starts with a `*' (or that
   (if (< arg 0)
       (beginning-of-line)
     (end-of-line))
-  (while (and (not (bobp)) (< arg 0))
-    (while (and (not (bobp))
-		(re-search-backward (concat "^\\(?:" outline-regexp "\\)")
-				    nil 'move)
-		(outline-invisible-p)))
-    (setq arg (1+ arg)))
-  (while (and (not (eobp)) (> arg 0))
-    (while (and (not (eobp))
-		(re-search-forward (concat "^\\(?:" outline-regexp "\\)")
-				   nil 'move)
-		(outline-invisible-p (match-beginning 0))))
-    (setq arg (1- arg)))
-  (beginning-of-line))
+  (let (found-heading-p)
+    (while (and (not (bobp)) (< arg 0))
+      (while (and (not (bobp))
+		  (setq found-heading-p
+			(re-search-backward
+			 (concat "^\\(?:" outline-regexp "\\)")
+			 nil 'move))
+		  (outline-invisible-p)))
+      (setq arg (1+ arg)))
+    (while (and (not (eobp)) (> arg 0))
+      (while (and (not (eobp))
+		  (setq found-heading-p
+			(re-search-forward
+			 (concat "^\\(?:" outline-regexp "\\)")
+			 nil 'move))
+		  (outline-invisible-p (match-beginning 0))))
+      (setq arg (1- arg)))
+    (if found-heading-p (beginning-of-line))))
 
 (defun outline-previous-visible-heading (arg)
   "Move to the previous heading line.