* bug#48294: Use 'with-current-buffer' byte-compiler warning seems wrong
@ 2021-05-08 20:14 Robert Weiner
2021-05-09 8:05 ` Eli Zaretskii
0 siblings, 1 reply; 6+ messages in thread
From: Robert Weiner @ 2021-05-08 20:14 UTC (permalink / raw)
To: 48294
[-- Attachment #1: Type: text/plain, Size: 1300 bytes --]
Hi:
I get a lot of these byte-compile warnings in my Elisp code:
hbut.el:683:26:Warning: Use ‘with-current-buffer’ rather than
save-excursion+set-buffer
but since with-current-buffer does not save the value of point, it is
not a valid substitute for save-excursion and should not be suggested.
Evaluate the two samples below and you will see that they are not
equivalent. If I am correct, I'd like this suggestion disabled. Thanks.
-- rsw
(save-excursion
(set-buffer (current-buffer))
(forward-char 20))
(with-current-buffer (current-buffer)
(forward-char 20))
------
In GNU Emacs 27.1.90 (build 1, x86_64-apple-darwin18.7.0, NS appkit-1671.60
Version 10.14.6 (Build 18G95))
of 2020-12-18 built on builder10-14.porkrind.org
Windowing system distributor 'Apple', version 10.3.1894
System Description: Mac OS X 10.15.7
Configured using:
'configure --with-ns '--enable-locallisppath=/Library/Application
Support/Emacs/${version}/site-lisp:/Library/Application
Support/Emacs/site-lisp' --with-modules'
Configured features:
NOTIFY KQUEUE ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS NS MODULES
THREADS JSON PDUMPER
Important settings:
value of $LANG: en_US.UTF-8
locale-coding-system: utf-8-unix
Major mode: Lisp Interaction
[-- Attachment #2: Type: text/html, Size: 1630 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread* bug#48294: Use 'with-current-buffer' byte-compiler warning seems wrong
2021-05-08 20:14 bug#48294: Use 'with-current-buffer' byte-compiler warning seems wrong Robert Weiner
@ 2021-05-09 8:05 ` Eli Zaretskii
2021-05-09 15:57 ` Robert Weiner
0 siblings, 1 reply; 6+ messages in thread
From: Eli Zaretskii @ 2021-05-09 8:05 UTC (permalink / raw)
To: rswgnu; +Cc: 48294
tags 48294 notabug
thanks
> From: Robert Weiner <rsw@gnu.org>
> Date: Sat, 8 May 2021 16:14:09 -0400
>
> I get a lot of these byte-compile warnings in my Elisp code:
>
> hbut.el:683:26:Warning: Use ‘with-current-buffer’ rather than
> save-excursion+set-buffer
>
> but since with-current-buffer does not save the value of point, it is
> not a valid substitute for save-excursion and should not be suggested.
> Evaluate the two samples below and you will see that they are not
> equivalent. If I am correct, I'd like this suggestion disabled. Thanks. -- rsw
>
> (save-excursion
> (set-buffer (current-buffer))
> (forward-char 20))
>
> (with-current-buffer (current-buffer)
> (forward-char 20))
The ELisp manual says about this:
Because ‘save-excursion’ only saves point for the buffer that was
current at the start of the excursion, any changes made to point in
other buffers, during the excursion, will remain in effect afterward.
This frequently leads to unintended consequences, so the byte compiler
warns if you call ‘set-buffer’ during an excursion:
Warning: Use ‘with-current-buffer’ rather than
save-excursion+set-buffer
To avoid such problems, you should call ‘save-excursion’ only after
setting the desired current buffer, as in the following example:
(defun append-string-to-buffer (string buffer)
"Append STRING to the end of BUFFER."
(with-current-buffer buffer
(save-excursion
(goto-char (point-max))
(insert string))))
I believe this example shows how to solve your problem.
^ permalink raw reply [flat|nested] 6+ messages in thread* bug#48294: Use 'with-current-buffer' byte-compiler warning seems wrong
2021-05-09 8:05 ` Eli Zaretskii
@ 2021-05-09 15:57 ` Robert Weiner
2021-05-09 16:11 ` Eli Zaretskii
0 siblings, 1 reply; 6+ messages in thread
From: Robert Weiner @ 2021-05-09 15:57 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: 48294
[-- Attachment #1: Type: text/plain, Size: 2812 bytes --]
Hi Eli:
Thanks for the feedback. Everything you wrote is very clear. The issue,
however, is that I want to save point in the current buffer prior to
switching context to the 'with-current-buffer' buffer, just as my
(save-excursion (set-buffer ...)) code does. If I move the save-excursion
into the with-current-buffer body, then it applies to the new buffer not
the original one and if in that same body we change buffers again to the
original and move point, then that movement will stay in place. So the
question is, what is the appropriate code that the byte-compiler will
accept when you want to save your original place before switching buffers.
Do I have to just do a (let ((opoint (point))) ...) and then restore it?
The issue is that within the with-current-buffer body, there could be a
hard to trace sequence of calls any of which could switch back to the
original buffer and move point. So how would you protect against that?
Thanks,
Bob
On Sun, May 9, 2021 at 4:05 AM Eli Zaretskii <eliz@gnu.org> wrote:
> tags 48294 notabug
> thanks
>
> > From: Robert Weiner <rsw@gnu.org>
> > Date: Sat, 8 May 2021 16:14:09 -0400
> >
> > I get a lot of these byte-compile warnings in my Elisp code:
> >
> > hbut.el:683:26:Warning: Use ‘with-current-buffer’ rather than
> > save-excursion+set-buffer
> >
> > but since with-current-buffer does not save the value of point, it is
> > not a valid substitute for save-excursion and should not be suggested.
> > Evaluate the two samples below and you will see that they are not
> > equivalent. If I am correct, I'd like this suggestion disabled.
> Thanks. -- rsw
> >
> > (save-excursion
> > (set-buffer (current-buffer))
> > (forward-char 20))
> >
> > (with-current-buffer (current-buffer)
> > (forward-char 20))
>
> The ELisp manual says about this:
>
> Because ‘save-excursion’ only saves point for the buffer that was
> current at the start of the excursion, any changes made to point in
> other buffers, during the excursion, will remain in effect afterward.
> This frequently leads to unintended consequences, so the byte compiler
> warns if you call ‘set-buffer’ during an excursion:
>
> Warning: Use ‘with-current-buffer’ rather than
> save-excursion+set-buffer
>
> To avoid such problems, you should call ‘save-excursion’ only after
> setting the desired current buffer, as in the following example:
>
> (defun append-string-to-buffer (string buffer)
> "Append STRING to the end of BUFFER."
> (with-current-buffer buffer
> (save-excursion
> (goto-char (point-max))
> (insert string))))
>
> I believe this example shows how to solve your problem.
>
[-- Attachment #2: Type: text/html, Size: 3961 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#48294: Use 'with-current-buffer' byte-compiler warning seems wrong
2021-05-09 15:57 ` Robert Weiner
@ 2021-05-09 16:11 ` Eli Zaretskii
2021-05-09 18:09 ` Robert Weiner
0 siblings, 1 reply; 6+ messages in thread
From: Eli Zaretskii @ 2021-05-09 16:11 UTC (permalink / raw)
To: rswgnu; +Cc: 48294
> From: Robert Weiner <rsw@gnu.org>
> Date: Sun, 9 May 2021 11:57:25 -0400
> Cc: 48294@debbugs.gnu.org
>
> Thanks for the feedback. Everything you wrote is very clear. The issue, however, is that I want to save
> point in the current buffer prior to switching context to the 'with-current-buffer' buffer, just as my
> (save-excursion (set-buffer ...)) code does. If I move the save-excursion into the with-current-buffer body,
> then it applies to the new buffer not the original one and if in that same body we change buffers again to the
> original and move point, then that movement will stay in place. So the question is, what is the appropriate
> code that the byte-compiler will accept when you want to save your original place before switching buffers.
I don't understand: with-current-buffer doesn't change point of the
original buffer, so why do you need to save it?
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#48294: Use 'with-current-buffer' byte-compiler warning seems wrong
2021-05-09 16:11 ` Eli Zaretskii
@ 2021-05-09 18:09 ` Robert Weiner
2021-05-09 18:24 ` Eli Zaretskii
0 siblings, 1 reply; 6+ messages in thread
From: Robert Weiner @ 2021-05-09 18:09 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: 48294
Somtimes a nested function within the body of the with-current-buffer will change the buffer and move point in the original buffer and this may largely only be known at run-time.
-- Bob
> On May 9, 2021, at 12:11 PM, Eli Zaretskii <eliz@gnu.org> wrote:
>
>
>>
>> From: Robert Weiner <rsw@gnu.org>
>> Date: Sun, 9 May 2021 11:57:25 -0400
>> Cc: 48294@debbugs.gnu.org
>>
>> Thanks for the feedback. Everything you wrote is very clear. The issue, however, is that I want to save
>> point in the current buffer prior to switching context to the 'with-current-buffer' buffer, just as my
>> (save-excursion (set-buffer ...)) code does. If I move the save-excursion into the with-current-buffer body,
>> then it applies to the new buffer not the original one and if in that same body we change buffers again to the
>> original and move point, then that movement will stay in place. So the question is, what is the appropriate
>> code that the byte-compiler will accept when you want to save your original place before switching buffers.
>
> I don't understand: with-current-buffer doesn't change point of the
> original buffer, so why do you need to save it?
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#48294: Use 'with-current-buffer' byte-compiler warning seems wrong
2021-05-09 18:09 ` Robert Weiner
@ 2021-05-09 18:24 ` Eli Zaretskii
0 siblings, 0 replies; 6+ messages in thread
From: Eli Zaretskii @ 2021-05-09 18:24 UTC (permalink / raw)
To: Robert Weiner; +Cc: 48294
> From: Robert Weiner <rswgnu@gmail.com>
> Date: Sun, 9 May 2021 14:09:23 -0400
> Cc: 48294@debbugs.gnu.org
>
> Somtimes a nested function within the body of the with-current-buffer will change the buffer and move point in the original buffer and this may largely only be known at run-time.
Then I'd use save-excursion outside of with-current-buffer.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-05-09 18:24 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-08 20:14 bug#48294: Use 'with-current-buffer' byte-compiler warning seems wrong Robert Weiner
2021-05-09 8:05 ` Eli Zaretskii
2021-05-09 15:57 ` Robert Weiner
2021-05-09 16:11 ` Eli Zaretskii
2021-05-09 18:09 ` Robert Weiner
2021-05-09 18:24 ` Eli Zaretskii
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).