* bug#14611: Suggest warning: forward-sexp not from program
@ 2013-06-13 16:14 Andreas Röhler
2013-06-13 16:41 ` Glenn Morris
0 siblings, 1 reply; 2+ messages in thread
From: Andreas Röhler @ 2013-06-13 16:14 UTC (permalink / raw)
To: 14611
Hi,
as `forward-sexp-function' might be set by users, `forward-sexp' is susceptible to fail when built into a program - which executes then set new binding.
Just came across that mistake.
Suggest a warning in it's docstring, saying something like: "Don't use..."
While pointing at core-function `scan-sexp' resp. `forward-sexp-function'.
Thanks,
Andreas
^ permalink raw reply [flat|nested] 2+ messages in thread
* bug#14611: Suggest warning: forward-sexp not from program
2013-06-13 16:14 bug#14611: Suggest warning: forward-sexp not from program Andreas Röhler
@ 2013-06-13 16:41 ` Glenn Morris
0 siblings, 0 replies; 2+ messages in thread
From: Glenn Morris @ 2013-06-13 16:41 UTC (permalink / raw)
To: 14611-done
Andreas Röhler wrote:
> as `forward-sexp-function' might be set by users, `forward-sexp' is
> susceptible to fail when built into a program - which executes then
> set new binding. Just came across that mistake.
So let-bind forward-sexp-function around any calls where you think that
might happen.
forward-sexp is user in 100s of places in lisp/.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-06-13 16:41 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-13 16:14 bug#14611: Suggest warning: forward-sexp not from program Andreas Röhler
2013-06-13 16:41 ` Glenn Morris
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).