[Top][All Lists]

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

safe-local-variable additions (was: Risky local variable mechanism)

From: Reiner Steib
Subject: safe-local-variable additions (was: Risky local variable mechanism)
Date: Thu, 16 Feb 2006 15:02:10 +0100
User-agent: Gnus/5.110004 (No Gnus v0.4) Emacs/22.0.50 (gnu/linux)

On Tue, Feb 14 2006, Chong Yidong wrote:

> I've installed the patch changing the way file local variables are
> handled.  Please see if there are any problems.
> If you encounter any file variables that should be marked safe, so
> that Emacs in its default configuration doesn't bother you before
> setting it, go ahead and add an appropriate `safe-local-variable'
> property.

`truncate-lines' could be added.  Any objection against adding it?

Is adding it to the "Commonly-encountered local variables that are
safe:" expression in `files.el' the right way to do this?

--8<---------------cut here---------------start------------->8---
--- files.el    15 Feb 2006 16:04:59 +0100      1.811
+++ files.el    15 Feb 2006 16:38:42 +0100      
@@ -2563,6 +2563,7 @@
            (sentence-end       . ,string-or-null)
            (sentence-end-double-space . t)
            (tab-width          .  integerp)
+           (truncate-lines     .  t)
            (version-control    .  t)))))
 (defun safe-local-variable-p (sym val)
--8<---------------cut here---------------end--------------->8---

`ispell-check-comments' and `ispell-local-dictionary' should also be
added, IMHO.

Is it better to put them in `files.el' or add it to `ispell.el' right
after the corresponding `defcustom'?

--8<---------------cut here---------------start------------->8---
(put 'ispell-check-comments 'safe-local-variable
     (lambda (a) (or (null a) (eq t a) (eq 'exclusive a))))

(put 'ispell-local-dictionary 'safe-local-variable
     (lambda (a) (or (stringp a) (null a))))
--8<---------------cut here---------------end--------------->8---

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]