bug#21886: 24.5; `dired-next/previous-line' should have a "^" in their interactive specs

[Eli Barzilay <eli@barzilay.org>]

[-- Attachment #1: message body text --]
[-- Type: text/plain, Size: 207 bytes --]

If I bind the up/down arrow keys to `dired-next/previous-line' in
dired mode, they misbehave wrt shift selection.  The following simple
patch fixes this by adding a "^" to their `interactive' declarations.


[-- Attachment #2: 0001-Add-to-the-interactive-specs-of-dired-next-previous-.patch --]
[-- Type: application/octet-stream, Size: 1297 bytes --]

From 195a4881482181443ffcec85e1863881c86edd35 Mon Sep 17 00:00:00 2001
From: Eli Barzilay <eli@barzilay.org>
Date: Thu, 12 Nov 2015 03:07:38 -0500
Subject: [PATCH] Add "^" to the interactive specs of
 `dired-next/previous-line'

It makes sense to bind these commands to the arrow keys, and that means
that they work better with a "^" in the `interactive' declaration so
selection works as expected.
---
 lisp/dired.el | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lisp/dired.el b/lisp/dired.el
index 5f0a83a..049d45d 100644
--- a/lisp/dired.el
+++ b/lisp/dired.el
@@ -2031,7 +2031,7 @@ Otherwise, toggle `read-only-mode'."
 (defun dired-next-line (arg)
   "Move down lines then position at filename.
 Optional prefix ARG says how many lines to move; default is one line."
-  (interactive "p")
+  (interactive "^p")
   (let ((line-move-visual)
 	(goal-column))
     (line-move arg t))
@@ -2044,7 +2044,7 @@ Optional prefix ARG says how many lines to move; default is one line."
 (defun dired-previous-line (arg)
   "Move up lines then position at filename.
 Optional prefix ARG says how many lines to move; default is one line."
-  (interactive "p")
+  (interactive "^p")
   (dired-next-line (- (or arg 1))))
 
 (defun dired-next-dirline (arg &optional opoint)
-- 
2.6.3


[-- Attachment #3: .signature --]
[-- Type: text/plain, Size: 151 bytes --]


-- 
                    ((x=>x(x))(x=>x(x)))                   Eli Barzilay:
                    http://barzilay.org/                   Maze is Life!