all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Juri Linkov <juri@jurta.org>
To: bug-gnu-emacs@gnu.org
Subject: bug#4554: local variable for updating the time stamp on save
Date: Thu, 24 Sep 2009 23:51:23 +0300	[thread overview]
Message-ID: <87hbut1vpi.fsf@mail.jurta.org> (raw)

The defcustom `safe-local-eval-forms' contains

    '((add-hook 'write-file-hooks 'time-stamp))

but the documentation suggests using

    (add-hook 'before-save-hook 'time-stamp)

This patch adds the recommended form to `safe-local-eval-forms' as well:

Index: lisp/files.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/files.el,v
retrieving revision 1.1083
diff -c -r1.1083 files.el
*** lisp/files.el	21 Sep 2009 22:12:30 -0000	1.1083
--- lisp/files.el	24 Sep 2009 20:47:28 -0000
***************
*** 2687,2693 ****
    :group 'find-file
    :type 'alist)
  
! (defcustom safe-local-eval-forms '((add-hook 'write-file-hooks 'time-stamp))
    "Expressions that are considered safe in an `eval:' local variable.
  Add expressions to this list if you want Emacs to evaluate them, when
  they appear in an `eval' local variable specification, without first
--- 2687,2695 ----
    :group 'find-file
    :type 'alist)
  
! (defcustom safe-local-eval-forms
!   '((add-hook 'before-save-hook 'time-stamp)
!     (add-hook 'write-file-hooks 'time-stamp))
    "Expressions that are considered safe in an `eval:' local variable.
  Add expressions to this list if you want Emacs to evaluate them, when
  they appear in an `eval' local variable specification, without first

-- 
Juri Linkov
http://www.jurta.org/emacs/






             reply	other threads:[~2009-09-24 20:51 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <87eipuistr.fsf@stupidchicken.com>
2009-09-24 20:51 ` Juri Linkov [this message]
2009-09-25  0:44   ` bug#4554: local variable for updating the time stamp on save Stefan Monnier
2009-09-25 22:15   ` bug#4554: marked as done (local variable for updating the time stamp on save) Emacs bug Tracking System

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87hbut1vpi.fsf@mail.jurta.org \
    --to=juri@jurta.org \
    --cc=4554@emacsbugs.donarmstrong.com \
    --cc=bug-gnu-emacs@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.