Better multi-line input support in comint.el

Revert the change of comint-line-beginning-position callers,
and modify comint-line-beginning-position instead.

* lisp/comint.el (comint-history-isearch-search)
(comint-history-isearch-message, comint-history-isearch-wrap):
Use comint-line-beginning-position instead of field-beginning.
(comint-send-input): Use either end-of-line or field-end
depending on comint-use-prompt-regexp.
(comint-line-beginning-position): Search backward
for comint-prompt-regexp if comint-use-prompt-regexp is non-nil.
Use field-beginning instead of line-beginning-position
if comint-use-prompt-regexp is nil.

Fixes: debbugs:19710

2 files changed, 29 insertions, 10 deletions

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 0e00cb2f17f..6c3caa826e8 100644
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,18 @@
+2015-02-05  Juri Linkov  <juri@linkov.net>
+
+        Revert the previous change of comint-line-beginning-position callers,
+        and modify comint-line-beginning-position instead.
+
+        * comint.el (comint-history-isearch-search)
+        (comint-history-isearch-message, comint-history-isearch-wrap):
+        Use comint-line-beginning-position instead of field-beginning.
+        (comint-send-input): Use either end-of-line or field-end
+        depending on comint-use-prompt-regexp.
+        (comint-line-beginning-position): Search backward
+        for comint-prompt-regexp if comint-use-prompt-regexp is non-nil.
+        Use field-beginning instead of line-beginning-position
+        if comint-use-prompt-regexp is nil.  (Bug#19710)
+
 2015-02-04  Robert Pluim  <rpluim@gmail.com>  (tiny change)
 
         * calendar/todo-mode.el (todo-item-done): When done items are
diff --git a/lisp/comint.el b/lisp/comint.el
index f33034bdb57..419938ea684 100644
--- a/lisp/comint.el
+++ b/lisp/comint.el
@@ -1475,7 +1475,7 @@ Intended to be added to `isearch-mode-hook' in `comint-mode'."
       (or
        ;; 1. First try searching in the initial comint text
        (funcall search-fun string
-                (if isearch-forward bound (field-beginning))
+                (if isearch-forward bound (comint-line-beginning-position))
                 noerror)
        ;; 2. If the above search fails, start putting next/prev history
        ;; elements in the comint successively, and search the string
@@ -1491,7 +1491,7 @@ Intended to be added to `isearch-mode-hook' in `comint-mode'."
                         (when (null comint-input-ring-index)
                           (error "End of history; no next item"))
                         (comint-next-input 1)
-                        (goto-char (field-beginning)))
+                        (goto-char (comint-line-beginning-position)))
                        (t
                         ;; Signal an error here explicitly, because
                         ;; `comint-previous-input' doesn't signal an error.
@@ -1509,7 +1509,7 @@ Intended to be added to `isearch-mode-hook' in `comint-mode'."
                                       (unless isearch-forward
                                         ;; For backward search, don't search
                                         ;; in the comint prompt
-                                        (field-beginning))
+                                        (comint-line-beginning-position))
                                       noerror)))
                ;; Return point of the new search result
                (point))
@@ -1533,16 +1533,16 @@ the function `isearch-message'."
     (if (overlayp comint-history-isearch-message-overlay)
         (move-overlay comint-history-isearch-message-overlay
                       (save-excursion
-                        (goto-char (field-beginning))
+                        (goto-char (comint-line-beginning-position))
                         (forward-line 0)
                         (point))
-                      (field-beginning))
+                      (comint-line-beginning-position))
       (setq comint-history-isearch-message-overlay
             (make-overlay (save-excursion
-                            (goto-char (field-beginning))
+                            (goto-char (comint-line-beginning-position))
                             (forward-line 0)
                             (point))
-                          (field-beginning)))
+                          (comint-line-beginning-position)))
       (overlay-put comint-history-isearch-message-overlay 'evaporate t))
     (overlay-put comint-history-isearch-message-overlay
                  'display (isearch-message-prefix ellipsis isearch-nonincremental))
@@ -1563,7 +1563,7 @@ or to the last history element for a backward search."
       (comint-goto-input (1- (ring-length comint-input-ring)))
     (comint-goto-input nil))
   (setq isearch-success t)
-  (goto-char (if isearch-forward (field-beginning) (point-max))))
+  (goto-char (if isearch-forward (comint-line-beginning-position) (point-max))))
 
 (defun comint-history-isearch-push-state ()
   "Save a function restoring the state of input history search.
@@ -1781,7 +1781,10 @@ Similarly for Soar, Scheme, etc."
       (widen)
       (let* ((pmark (process-mark proc))
              (intxt (if (>= (point) (marker-position pmark))
-                        (progn (if comint-eol-on-send (goto-char (field-end)))
+                        (progn (if comint-eol-on-send
+                                   (if comint-use-prompt-regexp
+                                       (end-of-line)
+                                     (goto-char (field-end))))
                                (buffer-substring pmark (point)))
                       (let ((copy (funcall comint-get-old-input)))
                         (goto-char pmark)
@@ -2260,6 +2263,7 @@ a buffer local variable."
   (if comint-use-prompt-regexp
       ;; Use comint-prompt-regexp
       (save-excursion
+        (re-search-backward comint-prompt-regexp nil t)
         (beginning-of-line)
         (comint-skip-prompt)
         (point))
@@ -2270,7 +2274,7 @@ a buffer local variable."
     ;; if there are two fields on a line, then the first one is the
     ;; prompt, and the second one is an input field, and is front-sticky
     ;; (as input fields should be).
-    (constrain-to-field (line-beginning-position) (line-end-position))))
+    (constrain-to-field (field-beginning) (line-end-position))))
 
 (defun comint-bol (&optional arg)
   "Go to the beginning of line, then skip past the prompt, if any.