[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#50387: Possible bug in Tramp or in completions
From: |
Gregory Heytings |
Subject: |
bug#50387: Possible bug in Tramp or in completions |
Date: |
Mon, 06 Sep 2021 20:55:15 +0000 |
Hi Michael,
* lisp/net/tramp.el (tramp-completion-handle-file-name-all-completions):
Do not return methods when the filename already contains a complete method.
Fixes bug#50387.
I don't see this implemented. You rather don't return methods when the
filename does not contain a possible method string at all.
Yet this is what is happening. You can try it yourself: emacs -Q, M-:
(require 'tramp), M-: (add-to-list 'completion-styles 'substring), C-x C-f
/scp: TAB. This will display "Sole completion", and it displays "Sole
completion" because "scp:" and "scpx:" are added in the completion
candiates in tramp-completion-handle-file-name-all-completions. You can
see this with
(message "method '%s' result '%s'" method (tramp-get-completion-methods m))
around "Possible methods". With C-x C-f /scp TAB, you will see
method ’scp’ result ’(nil nil nil nil nil nil nil nil nil nil nil nil nil
nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil /scpx: /scp:
nil nil nil nil nil)’
which is correct, but with C-x C-f /scp: TAB, you will see
method ’’ result ’(nil nil nil nil nil nil nil nil nil nil nil nil nil nil
nil nil nil nil nil nil nil nil nil nil nil nil nil nil /scpx: /scp: nil
nil nil nil nil)’
which is wrong.
;; Possible methods.
- (setq result
- (append result (tramp-get-completion-methods m)))))))
+ (unless (or (string-empty-p method)
+ (string-empty-p user)
+ (string-empty-p host))
The check for (string-empty-p user) and (string-empty-p host) wouldn't
be needed, you are in the else branch of (or user host). IOW, user and
host are already nil, nothing left to test for them.
Whoops, yes, indeed. Updated patch attached.
Improve-handling-of-non-default-completion-styles-wi.patch
Description: Text Data
- bug#50387: Possible bug in Tramp or in completions, (continued)
- bug#50387: Possible bug in Tramp or in completions, Gregory Heytings, 2021/09/05
- bug#50387: Possible bug in Tramp or in completions, Michael Albinus, 2021/09/06
- bug#50387: Possible bug in Tramp or in completions, Gregory Heytings, 2021/09/06
- bug#50387: Possible bug in Tramp or in completions, Michael Albinus, 2021/09/06
- bug#50387: Possible bug in Tramp or in completions, Gregory Heytings, 2021/09/06
- bug#50387: Possible bug in Tramp or in completions, Michael Albinus, 2021/09/06
- bug#50387: Possible bug in Tramp or in completions, Gregory Heytings, 2021/09/06
- bug#50387: Possible bug in Tramp or in completions, Michael Albinus, 2021/09/06
- bug#50387: Possible bug in Tramp or in completions, Gregory Heytings, 2021/09/06
- bug#50387: Possible bug in Tramp or in completions, Michael Albinus, 2021/09/06
- bug#50387: Possible bug in Tramp or in completions,
Gregory Heytings <=
- bug#50387: Possible bug in Tramp or in completions, Michael Albinus, 2021/09/07
- bug#50387: Possible bug in Tramp or in completions, Gregory Heytings, 2021/09/07
- bug#50387: Possible bug in Tramp or in completions, Gregory Heytings, 2021/09/07
- bug#50387: Possible bug in Tramp or in completions, Michael Albinus, 2021/09/07
- bug#50387: Possible bug in Tramp or in completions, Gregory Heytings, 2021/09/07
- bug#50387: Possible bug in Tramp or in completions, Michael Albinus, 2021/09/07
- bug#50387: Possible bug in Tramp or in completions, Gregory Heytings, 2021/09/07
- bug#50387: Possible bug in Tramp or in completions, Michael Albinus, 2021/09/07
- bug#50387: Possible bug in Tramp or in completions, Gregory Heytings, 2021/09/07
- bug#50387: Possible bug in Tramp or in completions, Gregory Heytings, 2021/09/07