(bug-reference-url-format): Mark as `safe-local-variable' if the value

is a string or a symbol with the property `bug-reference-url-format'.

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index a97a6e9cc7d9057651e76de69f11833271d87d41..3eba70f40afd545468adccf1d262ae648390fead 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,9 @@
+2010-04-28  Sam Steingold  <sds@gnu.org>
+
+       * progmodes/bug-reference.el (bug-reference-url-format): Mark as
+       `safe-local-variable' if the value is a string or a symbol with
+       the property `bug-reference-url-format'.
+
 2010-04-28  Chong Yidong  <cyd@stupidchicken.com>
 
        * progmodes/bug-reference.el (bug-reference-url-format): Revert

diff --git a/lisp/progmodes/bug-reference.el b/lisp/progmodes/bug-reference.el
index 462f978ab787e3e03cb62f99b5eb6c13e493242a..4897581252fe1ae457e5d9fa1b10881ac062e45f 100644 (file)
--- a/lisp/progmodes/bug-reference.el
+++ b/lisp/progmodes/bug-reference.el
@@ -48,10 +48,18 @@ It can use `match-string' to get parts matched against
  1. issue kind (bug, patch, rfe &c)
  2. issue number.
 
-There is no default setting for this, it must be set per file.")
+There is no default setting for this, it must be set per file.
+If you set it to a symbol in the file Local Variables section,
+you need to add a `bug-reference-url-format' property to it:
+\(put 'my-bug-reference-url-format 'bug-reference-url-format t)
+so that it is considered safe, see `enable-local-variables'.")
 
 ;;;###autoload
-(put 'bug-reference-url-format 'safe-local-variable 'stringp)
+(put 'bug-reference-url-format 'safe-local-variable
+     (lambda (s)
+       (or (stringp s)
+           (and (symbolp s)
+                (get s 'bug-reference-url-format)))))
 
 (defconst bug-reference-bug-regexp
   "\\([Bb]ug ?#\\|[Pp]atch ?#\\|RFE ?#\\|PR [a-z-+]+/\\)\\([0-9]+\\)"