Subject: Re: bug#74799: [PATCH] Clarify that save-place-file must be set before save-place-mode starts

From: Hong Xu <[email-masked]>
Date: Wed, 11 Dec 2024 12:41:32 -0800

On 2024-12-11 Wed 12:29 GMT-08, Eli Zaretskii <[email-masked]> wrote:

>> From: Hong Xu <[email-masked]>
>> Date: Wed, 11 Dec 2024 12:00:57 -0800
>> 
>> ---
>>  lisp/saveplace.el | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>> 
>> diff --git a/lisp/saveplace.el b/lisp/saveplace.el
>> index 012e305f7f45..ea67c4567b63 100644
>> --- a/lisp/saveplace.el
>> +++ b/lisp/saveplace.el
>> @@ -63,7 +63,8 @@ save-place-alist
>>  This alist is saved between Emacs sessions.")
>>  
>>  (defcustom save-place-file (locate-user-emacs-file "places" ".emacs-places")
>> -  "Name of the file that records `save-place-alist' value."
>> +  "Name of the file that records `save-place-alist' value.
>> +Must be set before starting save-place-mode."
>>    :version "24.4"                       ; added locate-user-emacs-file
>>    :type 'file)
>
> Thanks, but how about removing the limitation instead?

I'm not sure how to do this. It seems to me the limitation didn't exist
in Emacs 29 but suddenly I have to set save-place-file before calling
save-place-mode. I thought this was intended but maybe this is a
regression?

(It's difficult for me to try out Emacs 29 now to 100% confirm, sorry.)

-- 
Hong