bug#59290: 29.0.50; [PATCH] Buffer-localize temporary-goal-column

[Stefan Kangas <stefankangas@gmail.com>]

Eli Zaretskii <eliz@gnu.org> writes:

>> From: dick <dick.r.chiang@gmail.com>
>> Date: Tue, 15 Nov 2022 16:52:10 -0500
>>
>> emacs -Q -l cl-lib --eval " \
>> (with-temp-buffer \
>>   (save-excursion \
>>     (insert \"עִבְרִית\nעִבְרִית\")) \
>>   (call-interactively (function next-line)) \
>>   (with-temp-buffer \
>>     (save-excursion (insert \"English\nEnglish\")) \
>>     (call-interactively (function next-line)) \
>>     (cl-assert (zerop (current-column)))))"
>>
>>
>> >From 44ca27270bdd7786ddb36469149128b6d18aac9a Mon Sep 17 00:00:00 2001
>> From: dickmao <dick.r.chiang@gmail.com>
>> Date: Tue, 15 Nov 2022 16:03:48 -0500
>> Subject: [PATCH] Temporary goal column leaks
>>
>> * lisp/simple.el (temporary-goal-column): Buffer-localize.
>> * test/lisp/simple-tests.el (line-move-visual-goal-column-leakage):
>> Test it.
>
> Thanks, but please provide a meaningful commit log message with an
> explanation of the problem and the solution.  AFAIU, it isn't
> self-evident.  For example, if I invoke the program interactively in a
> running Emacs session, the problem doesn't happen.  Your test seems to
> acknowledge that, as is evidenced by this:
>
>> +(ert-deftest line-move-visual-goal-column-leakage ()
>> +  (skip-unless (not noninteractive))  <<<<<<<<<<<<<<<<<<<<<<<
>
> but without any explanation.
>
> With the root cause of this stated explicitly, it might be that we
> will want or at least consider a different solution.

dick, could you please provide the information Eli is asking for above?

Thanks in advance.