all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* at risk
@ 2013-06-15 11:54 Andreas Röhler
  2013-06-17 15:45 ` Davis Herring
  0 siblings, 1 reply; 5+ messages in thread
From: Andreas Röhler @ 2013-06-15 11:54 UTC (permalink / raw)
  To: Emacs developers

When execution might rely on variables set by user, resp. at a later time, such a function should preferably not be used by programs, i.e.
in circumstances being with some probability not aware of the changed proceeding.

This is the case for example with `forward-sexp', which might be taken by `forward-sexp-function'.

Please consider bug#14611. Being concerned a certain amount of code is set already at risk that way.

Thanks,

Andreas









^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: at risk
  2013-06-15 11:54 at risk Andreas Röhler
@ 2013-06-17 15:45 ` Davis Herring
  2013-06-17 17:56   ` Andreas Röhler
  0 siblings, 1 reply; 5+ messages in thread
From: Davis Herring @ 2013-06-17 15:45 UTC (permalink / raw)
  To: Andreas Röhler; +Cc: Emacs developers

> This is the case for example with `forward-sexp', which might be taken
> by `forward-sexp-function'.

...which is not a user option?  It's meant for a major mode to set
(precisely so that other code that uses `forward-sexp' works
transparently), not the user.

Davis

-- 
This product is sold by volume, not by mass.  If it appears too dense or
too sparse, it is because mass-energy conversion has occurred during
shipping.



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: at risk
  2013-06-17 15:45 ` Davis Herring
@ 2013-06-17 17:56   ` Andreas Röhler
  2013-06-17 19:45     ` Davis Herring
  0 siblings, 1 reply; 5+ messages in thread
From: Andreas Röhler @ 2013-06-17 17:56 UTC (permalink / raw)
  To: Davis Herring; +Cc: Emacs developers

Am 17.06.2013 17:45, schrieb Davis Herring:
>> This is the case for example with `forward-sexp', which might be taken
>> by `forward-sexp-function'.
>
> ...which is not a user option?  It's meant for a major mode to set
> (precisely so that other code that uses `forward-sexp' works
> transparently), not the user.
>
> Davis
>

If a major-mode sets this for some purpose - how to exclude the Emacs core, who uses this and should proceed the common way, not as set lately?

Andreas



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: at risk
  2013-06-17 17:56   ` Andreas Röhler
@ 2013-06-17 19:45     ` Davis Herring
  2013-06-18  5:12       ` Andreas Röhler
  0 siblings, 1 reply; 5+ messages in thread
From: Davis Herring @ 2013-06-17 19:45 UTC (permalink / raw)
  To: Andreas Röhler; +Cc: Emacs developers

> If a major-mode sets this for some purpose - how to exclude the Emacs
> core, who uses this and should proceed the common way, not as set lately?

Use `scan-sexps', which is the majority of the default implementation of
`forward-sexp', instead?

Davis

-- 
This product is sold by volume, not by mass.  If it appears too dense or
too sparse, it is because mass-energy conversion has occurred during
shipping.



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: at risk
  2013-06-17 19:45     ` Davis Herring
@ 2013-06-18  5:12       ` Andreas Röhler
  0 siblings, 0 replies; 5+ messages in thread
From: Andreas Röhler @ 2013-06-18  5:12 UTC (permalink / raw)
  To: Davis Herring; +Cc: Emacs developers

Am 17.06.2013 21:45, schrieb Davis Herring:
>> If a major-mode sets this for some purpose - how to exclude the Emacs
>> core, who uses this and should proceed the common way, not as set lately?
>
> Use `scan-sexps', which is the majority of the default implementation of
> `forward-sexp', instead?#

Right, for example. For sure: forward-sexp is nicely designed as a command to tweak by users, that's a kind of stuff I like at Emacs.
Just avoid turning it into a bug-source, don't build in forward-sexp in Emacs Lisp core programs, as this answer indicates:

http://lists.gnu.org/archive/html/bug-gnu-emacs/2013-06/msg00338.html

Andreas
>
> Davis
>





^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-06-18  5:12 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-15 11:54 at risk Andreas Röhler
2013-06-17 15:45 ` Davis Herring
2013-06-17 17:56   ` Andreas Röhler
2013-06-17 19:45     ` Davis Herring
2013-06-18  5:12       ` Andreas Röhler

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.