unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#15820: 24.3.50; (elisp) `File Local Variables'
@ 2013-11-06 17:09 Drew Adams
  2013-11-06 18:01 ` Glenn Morris
  0 siblings, 1 reply; 4+ messages in thread
From: Drew Adams @ 2013-11-06 17:09 UTC (permalink / raw)
  To: 15820

This part of the text is unclear:

  When defining a user option using `defcustom', you can set its
  `safe-local-variable' property by adding the arguments `:safe FUNCTION'
  to `defcustom'

FUNCTION needs to be described here.  Just say that FUNCTION is the same
as the function used as the value of property `safe-local-variable'.

That will provide readers the understanding that FUNCTION "has to be a
function of one argument; any value is safe if the function returns
non-`nil' given that value."

In GNU Emacs 24.3.50.1 (i686-pc-mingw32)
 of 2013-10-19 on LEG570
Bzr revision: 114715 rgm@gnu.org-20131019023520-s8mwtib7xcx9e05w
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --enable-checking 'CFLAGS=-O0 -g3' CPPFLAGS=-DGLYPH_DEBUG=1'





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

* bug#15820: 24.3.50; (elisp) `File Local Variables'
  2013-11-06 17:09 bug#15820: 24.3.50; (elisp) `File Local Variables' Drew Adams
@ 2013-11-06 18:01 ` Glenn Morris
  2013-11-06 18:17   ` Drew Adams
  2013-11-06 19:56   ` Michael Heerdegen
  0 siblings, 2 replies; 4+ messages in thread
From: Glenn Morris @ 2013-11-06 18:01 UTC (permalink / raw)
  To: 15820

Drew Adams wrote:

> This part of the text is unclear:
>
>   When defining a user option using `defcustom', you can set its
>   `safe-local-variable' property by adding the arguments `:safe FUNCTION'
>   to `defcustom'
>
> FUNCTION needs to be described here.  Just say that FUNCTION is the same
> as the function used as the value of property `safe-local-variable'.

It is explained in the immediately previous paragraph.
Repeating it seems unnecessarily verbose, and we all hate unnecessary
verbosity, don't we?





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

* bug#15820: 24.3.50; (elisp) `File Local Variables'
  2013-11-06 18:01 ` Glenn Morris
@ 2013-11-06 18:17   ` Drew Adams
  2013-11-06 19:56   ` Michael Heerdegen
  1 sibling, 0 replies; 4+ messages in thread
From: Drew Adams @ 2013-11-06 18:17 UTC (permalink / raw)
  To: Glenn Morris, 15820

> > This part of the text is unclear:
> >
> >   When defining a user option using `defcustom', you can set its
> >   `safe-local-variable' property by adding the arguments `:safe
> >   FUNCTION' to `defcustom'
> >
> > FUNCTION needs to be described here.  Just say that FUNCTION is
> > the same as the function used as the value of property
> > `safe-local-variable'.
> 
> It is explained in the immediately previous paragraph.

FUNCTION is not described there.  What is described there is the
function that is the value of property `safe-local-variable'.

But I was mistaken that the connection between the two is not made
clear.  They are connected in the same sentence, by saying that
:safe FUNCTION _sets_ the property.  It is reasonable to assume
that FUNCTION is used as the value that is set.  Not a bug.  Thx.





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

* bug#15820: 24.3.50; (elisp) `File Local Variables'
  2013-11-06 18:01 ` Glenn Morris
  2013-11-06 18:17   ` Drew Adams
@ 2013-11-06 19:56   ` Michael Heerdegen
  1 sibling, 0 replies; 4+ messages in thread
From: Michael Heerdegen @ 2013-11-06 19:56 UTC (permalink / raw)
  To: Glenn Morris; +Cc: 15820

Glenn Morris <rgm@gnu.org> writes:

> It is explained in the immediately previous paragraph.
> Repeating it seems unnecessarily verbose, and we all hate unnecessary
> verbosity, don't we?

Don't be so schnippisch!


Regards,

Michael.





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

end of thread, other threads:[~2013-11-06 19:56 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-06 17:09 bug#15820: 24.3.50; (elisp) `File Local Variables' Drew Adams
2013-11-06 18:01 ` Glenn Morris
2013-11-06 18:17   ` Drew Adams
2013-11-06 19:56   ` Michael Heerdegen

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).