GNU bug report logs - #14611
Suggest warning: forward-sexp not from program

Previous Next

Package: emacs;

Reported by: Andreas Röhler <andreas.roehler <at> easy-emacs.de>

Date: Thu, 13 Jun 2013 16:15:02 UTC

Severity: wishlist

Tags: notabug, wontfix

Done: Glenn Morris <rgm <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Andreas Röhler <andreas.roehler <at> easy-emacs.de>
Subject: bug#14611: closed (Re: bug#14611: Suggest warning: forward-sexp
 not from program)
Date: Thu, 13 Jun 2013 16:42:03 +0000
[Message part 1 (text/plain, inline)]
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 14611 <at> debbugs.gnu.org.

-- 
14611: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=14611
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Glenn Morris <rgm <at> gnu.org>
To: 14611-done <at> debbugs.gnu.org
Subject: Re: bug#14611: Suggest warning: forward-sexp not from program
Date: Thu, 13 Jun 2013 12:41:18 -0400
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/.

[Message part 3 (message/rfc822, inline)]
From: Andreas Röhler <andreas.roehler <at> easy-emacs.de>
To: bug-gnu-emacs <at> gnu.org
Subject: Suggest warning: forward-sexp not from program
Date: Thu, 13 Jun 2013 18:14:54 +0200
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



This bug report was last modified 11 years and 348 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.