GNU bug report logs - #24409
25.1.1; dired-jump: Interactive call w/ prefix arg move to wrong file line

Previous Next

Full log

Message #20 received at 24409 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Tino Calancha <tino.calancha <at> gmail.com>
Cc: 24409 <at> debbugs.gnu.org
Subject: Re: bug#24409: 25.1.1; dired-jump: Interactive call w/ prefix arg
 move to wrong file line
Date: Sun, 11 Sep 2016 20:16:51 +0300

> From: Tino Calancha <tino.calancha <at> gmail.com>
> Date: Mon, 12 Sep 2016 02:03:14 +0900 (JST)
> cc: Tino Calancha <tino.calancha <at> gmail.com>, 24409 <at> debbugs.gnu.org
> 
> *) Please, take a look in the new patch below with more verbose log 
> message/comments.

Thanks, see a follow-up comment below.

> *) Where should go the fix: emacs-25 branch or master?

Master, of course.

> -    (let* ((file (or file-name buffer-file-name))
> +    ;; Expand file-name before calling `dired-goto-file':
> +    ;; This is required in interactive calls because the return value
> +    ;; of `read-file-name' is not expanded; it also helps in
> +    ;; non-interactive calls, where file-name could be an abbreviated
> +    ;; file name as ~/dir/file (Bug#24409).
> +    (let* ((file (or (and file-name (expand-file-name file-name))
> +                     buffer-file-name))

I think a better comment would be to say that dired-goto-file requires
its argument to be an absolute file name, and the result of
read-file-name could be an abbreviated file name.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.