GNU bug report logs - #524
outdated (put ...) exceptions for evaled file-local variables

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (Emacs bug Tracking System)
To: Chong Yidong <cyd <at> stupidchicken.com>
Subject: bug#524: marked as done (outdated (put ...) exceptions for evaled
 file-local variables)

[Message part 1 (text/plain, inline)]

Your message dated Tue, 05 Aug 2008 16:47:51 -0400
with message-id <878wvbqjd4.fsf <at> stupidchicken.com>
and subject line Re: outdated (put ...) exceptions for evaled file-local variables
has caused the Emacs bug report #524,
regarding outdated (put ...) exceptions for evaled file-local variables
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact don <at> donarmstrong.com
immediately.)


-- 
524: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=524
Emacs Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Alex Shinn <alexshinn <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: outdated (put ...) exceptions for evaled file-local variables
Date: Thu, 03 Jul 2008 15:44:53 +0900

[Message part 3 (text/plain, inline)]

Currently `hack-one-local-variable-eval-safep' has a few
manually checked exceptions for the `eval' pseudo-variable,
so that the common cases of setting indent and edebug
properties for symbols is considered safe automatically.

However, the indent property checks for `lisp-indent-hook',
which as far as I can tell is deprecated.  It's not
mentioned anywhere in the docs, and lisp-mode.el only uses
it as a fallback in case the `lisp-indent-function' property
isn't set.  All the properties in lisp-mode.el are set as
`lisp-indent-function'.

A simple fix would be to allow either.  I've attached a
potential patch that does this, and also allows
scheme-indent-function, since this case is also common and I
don't think there are many other modes that make use of
symbol indent-function properties.  I'd understand if you
didn't want to support Scheme by default though, in which
case I'd just resort to using `safe-local-eval-function'.

A more general fix might be to implement a separate
customization for which properties are automatically
settable with which values, with perhaps more broad default
settings than Emacs currently has.

-- 
Alex

[files.el.diff (text/x-patch, attachment)]

[Message part 5 (message/rfc822, inline)]

From: Chong Yidong <cyd <at> stupidchicken.com>
To: Alex Shinn <alexshinn <at> gmail.com>
Cc: 524-done <at> debbugs.gnu.org
Subject: Re: outdated (put ...) exceptions for evaled file-local variables
Date: Tue, 05 Aug 2008 16:47:51 -0400

Thanks.  I've checked your patch (with minor modifications) into the CVS
trunk.

Previous Next

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