GNU bug report logs - #15010
24.3; mark more checkdoc options as 'safe-local-variable

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 15010 in the body.
You can then email your comments to 15010 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Jörg Walter <info <at> syntax-k.de>
To: bug-gnu-emacs <at> gnu.org
Subject: 24.3; mark more checkdoc options as 'safe-local-variable
Date: Fri, 02 Aug 2013 21:42:56 +0200

Hi,

I suggest that in checkdoc.el, some more variables can be set as 'safe-local-
variable by default. Right now I use these additions:

     (put 'checkdoc-verb-check-experimental-flag 'safe-local-variable 
'booleanp)

     (put 'checkdoc-spellcheck-documentation-flag 'safe-local-variable 
'booleanp)
     (put 'checkdoc-ispell-list-words 'safe-local-variable 'checkdoc-list-of-
strings-p)

     (put 'checkdoc-proper-noun-regexp 'safe-local-variable 'stringp)
     (put 'checkdoc-common-verbs-regexp 'safe-local-variable 'stringp)

Having these as file-local variable by default would be a major convenience 
when doing batch checking of Lisp files. Right now I need to include the above 
statements on the command line.

-- 
Mit freundlichen Grüßen,

	Jörg Walter

Message #8 received at 15010 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Jörg Walter <info <at> syntax-k.de>
Cc: 15010 <at> debbugs.gnu.org
Subject: Re: bug#15010: 24.3;
 mark more checkdoc options as 'safe-local-variable
Date: Sun, 04 Aug 2013 17:34:26 -0400

> I suggest that in checkdoc.el, some more variables can be set as 'safe-local-
> variable by default.  Right now I use these additions:

>      (put 'checkdoc-verb-check-experimental-flag 'safe-local-variable 
> 'booleanp)

>      (put 'checkdoc-spellcheck-documentation-flag 'safe-local-variable 
> 'booleanp)
>      (put 'checkdoc-ispell-list-words 'safe-local-variable 'checkdoc-list-of-
> strings-p)

>      (put 'checkdoc-proper-noun-regexp 'safe-local-variable 'stringp)
>      (put 'checkdoc-common-verbs-regexp 'safe-local-variable 'stringp)

Could you describe the context in which you want these to be set
locally to particular files?


        Stefan

Message #13 received at 15010-done <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Jörg Walter <info <at> syntax-k.de>
Cc: 15010-done <at> debbugs.gnu.org
Subject: Re: bug#15010: 24.3;
 mark more checkdoc options as 'safe-local-variable
Date: Thu, 08 Aug 2013 21:21:13 -0400

> I suggest that in checkdoc.el, some more variables can be set as 'safe-local-
> variable by default. Right now I use these additions:

>      (put 'checkdoc-verb-check-experimental-flag 'safe-local-variable 
> 'booleanp)

>      (put 'checkdoc-spellcheck-documentation-flag 'safe-local-variable 
> 'booleanp)
>      (put 'checkdoc-ispell-list-words 'safe-local-variable 'checkdoc-list-of-
> strings-p)

>      (put 'checkdoc-proper-noun-regexp 'safe-local-variable 'stringp)
>      (put 'checkdoc-common-verbs-regexp 'safe-local-variable 'stringp)

OK, added.

I really think this is just a stop-gap, tho.


        Stefan

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.