emacs-pretest-bug
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: add info about safe-local-variable to describe-variable


From: Reiner Steib
Subject: Re: add info about safe-local-variable to describe-variable
Date: Fri, 12 May 2006 18:34:20 +0200
User-agent: Gnus/5.110006 (No Gnus v0.6) Emacs/22.0.50 (gnu/linux)

On Fri, May 12 2006, Ken Manheimer wrote:

> i've attached a patch (allout-patch.txt), ChangeLog
> (allout-ChangeLog.txt), and revised NEWS fragment (allout-NEWS.txt)
> for allout that addresses the stuff we've been discussing, and a few
> other things.

Thanks, I have committed your changes.

> in the patch, any autoloaded safe-local-variables lambdas are quoted.
> the patch also incorporates some of the other refinements that were
> suggested in this discussion, though i am leaving the
> safe-local-variable property assignments as active code (rather than
> containing them in the autoload comment) 

A drawback is that user's won't see the predicate in the output of
`describe-variable':

,----
| This variable is safe as a file local variable if its value
| satisfies the predicate which is byte-compiled expression.
`----

> and continuing to use a lambda rather than booleanp.  both of these
> measures will make it easier to use allout in older emacs, without
> adding significant complexity.

But here, I still don't think it makes much sense to provide backward
compatibility for older CVS Emacs versions.  Not using `booleanp'
makes the code and the output of `describe-variable' harder to read.

If you can't update your CVS Emacs, you could define `booleanp' if
it's not bound.

Bye, Reiner.
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]