Subject: bug#14611: closed (Re: bug#14611: Suggest warning: forward-sexp not from program)

From: [email-masked] (GNU bug Tracking System)
Date: Thu, 13 Jun 2013 16:42:03 +0000

Your bug report

#14611: Suggest warning: forward-sexp not from program

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
14611: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=14611
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#14611: Suggest warning: forward-sexp not from program
From: Glenn Morris <[email-masked]>
To: [email-masked]
Date: Thu, 13 Jun 2013 12:41:18 -0400
List-Id: <debbugs-submit.debbugs.gnu.org>

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/.

Subject: Suggest warning: forward-sexp not from program
From: Andreas Röhler <[email-masked]>
To: [email-masked]
Date: Thu, 13 Jun 2013 18:14:54 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

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