GNU bug report logs - #66150
30.0.50; Duplicate :tag in remote-file-name-inhibit-cache

Previous Next

Full log

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

From: "Basil L. Contovounesios" <contovob <at> tcd.ie>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 66150-done <at> debbugs.gnu.org, michael.albinus <at> gmx.de,
 Miha Rihtaršič <miha <at> kamnitnik.top>
Subject: Re: bug#66150: 30.0.50; Duplicate :tag in
 remote-file-name-inhibit-cache
Date: Fri, 22 Sep 2023 15:19:35 +0200

close 66150 29.1
quit

Eli Zaretskii [2023-09-22 14:37 +0300] wrote:

>> Cc: Michael Albinus <michael.albinus <at> gmx.de>
>> Date: Fri, 22 Sep 2023 11:41:32 +0200
>> From:  "Basil L. Contovounesios" via "Bug reports for GNU Emacs,
>>  the Swiss army knife of text editors" <bug-gnu-emacs <at> gnu.org>
>> 
>> -	  (const   :tag "Do not inhibit file name cache" nil)
>> -	  (const   :tag "Do not use file name cache" t)
>> -	  (integer :tag "Do not use file name cache"
>> -		   :format "Do not use file name cache older then %v seconds"
>> +          (const   :tag "Inhibit remote file cache" t)
>> +          (const   :tag "Cache remote file attributes" nil)
>> +          (integer :tag "Cache remote file attributes with expiration"
>> +                   :format "Cache expiry in seconds: %v"
>>  		   :value 10)))
>
> Let's be consistent.  Either (a) the t value should have the tag of
> "Do not cache remote file attributes", or (b) the nil and the integer
> values should have the tag "Do not inhibit remote file cache" and
> "Inhibit file cache if expired", respectively.  OK?

Of course, thanks.  I went with option (a) for both
remote-file-name-inhibit-cache and
shell-highlight-undef-remote-file-name-inhibit-cache.

Installed on emacs-29, and closing.

Improve remote-file-name-inhibit-cache :type
e27ec0e4144 2023-09-22 15:15:36 +0200
https://git.sv.gnu.org/cgit/emacs.git/commit/?id=e27ec0e4144

-- 
Basil

Previous Next

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