all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Robert Pluim <rpluim@gmail.com>
To: emacs-devel@gnu.org
Cc: Michael Albinus <michael.albinus@gmx.de>
Subject: Re: master b34f9a44393: Apply remote-file-name-access-timeout in desktop.el and recentf.el
Date: Tue, 04 Jul 2023 16:16:23 +0200	[thread overview]
Message-ID: <87jzvf92yw.fsf@gmail.com> (raw)
In-Reply-To: <20230704132719.3251EC11DCA@vcs2.savannah.gnu.org> (Michael Albinus's message of "Tue, 4 Jul 2023 09:27:18 -0400 (EDT)")

>>>>> On Tue,  4 Jul 2023 09:27:18 -0400 (EDT), Michael Albinus <Michael.Albinus@gmx.de> said:

    Michael> branch: master
    Michael> commit b34f9a44393a331c866268d93b16ad69567a3cd4
    Michael> Author: Michael Albinus <michael.albinus@gmx.de>
    Michael> Commit: Michael Albinus <michael.albinus@gmx.de>

    Michael>     Apply remote-file-name-access-timeout in desktop.el and recentf.el
    
    Michael>     * doc/emacs/files.texi (File Conveniences):
    Michael>     * doc/emacs/misc.texi (Saving Emacs Sessions):
    Michael>     Mention remote-file-name-access-timeout.
    
    Michael>     * doc/misc/tramp.texi (Frequently Asked Questions): Add note.
    
    Michael>     * etc/NEWS: Explain use of 'remote-file-name-access-timeout' in
    Michael>     desktop.el and recentf.el.
    
    Michael>     * lisp/desktop.el (desktop-access-file): New defun.
    Michael>     (desktop-restore-file-buffer): Use it.
    
    Michael>     * lisp/recentf.el (recentf-access-file): New defun.
    Michael>     (recentf-keep-default-predicate): Use it.

This is a good change, although I wonder if we should set
`remote-file-name-access-timeout' non-nil by default, maybe something
relatively conservative like 60 seconds?

(I also see you use `natnump' for
`remote-file-name-access-timeout'. Is 0 a sane value to allow, I think
it would cause immediate timeout?)

Robert
-- 



       reply	other threads:[~2023-07-04 14:16 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <168847723857.12043.8020935461264507935@vcs2.savannah.gnu.org>
     [not found] ` <20230704132719.3251EC11DCA@vcs2.savannah.gnu.org>
2023-07-04 14:16   ` Robert Pluim [this message]
2023-07-04 14:42     ` master b34f9a44393: Apply remote-file-name-access-timeout in desktop.el and recentf.el Michael Albinus
2023-07-04 15:41       ` Robert Pluim
2023-07-04 16:29         ` Eli Zaretskii
2023-07-04 17:28           ` Robert Pluim
2023-07-04 16:40         ` Michael Albinus

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87jzvf92yw.fsf@gmail.com \
    --to=rpluim@gmail.com \
    --cc=emacs-devel@gnu.org \
    --cc=michael.albinus@gmx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.