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/ _______________________________________________ emacs-pretest-bug mailing list emacs-pretest-bug@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug