unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#15010: 24.3; mark more checkdoc options as 'safe-local-variable
@ 2013-08-02 19:42 Jörg Walter
  2013-08-04 21:34 ` Stefan Monnier
  2013-08-09  1:21 ` Stefan Monnier
  0 siblings, 2 replies; 3+ messages in thread
From: Jörg Walter @ 2013-08-02 19:42 UTC (permalink / raw)
  To: 15010

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





^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#15010: 24.3; mark more checkdoc options as 'safe-local-variable
  2013-08-02 19:42 bug#15010: 24.3; mark more checkdoc options as 'safe-local-variable Jörg Walter
@ 2013-08-04 21:34 ` Stefan Monnier
  2013-08-09  1:21 ` Stefan Monnier
  1 sibling, 0 replies; 3+ messages in thread
From: Stefan Monnier @ 2013-08-04 21:34 UTC (permalink / raw)
  To: Jörg Walter; +Cc: 15010

> 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





^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#15010: 24.3; mark more checkdoc options as 'safe-local-variable
  2013-08-02 19:42 bug#15010: 24.3; mark more checkdoc options as 'safe-local-variable Jörg Walter
  2013-08-04 21:34 ` Stefan Monnier
@ 2013-08-09  1:21 ` Stefan Monnier
  1 sibling, 0 replies; 3+ messages in thread
From: Stefan Monnier @ 2013-08-09  1:21 UTC (permalink / raw)
  To: Jörg Walter; +Cc: 15010-done

> 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





^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-08-09  1:21 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-02 19:42 bug#15010: 24.3; mark more checkdoc options as 'safe-local-variable Jörg Walter
2013-08-04 21:34 ` Stefan Monnier
2013-08-09  1:21 ` Stefan Monnier

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).